Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: installer/src/installer-lib/test/test-installer-lib-sandbox.cpp

Issue 5663180676136960: Make sure IE and Engine are actually being closed by custom action (Closed)
Patch Set: Created June 20, 2014, 10:33 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: installer/src/installer-lib/test/test-installer-lib-sandbox.cpp
===================================================================
--- a/installer/src/installer-lib/test/test-installer-lib-sandbox.cpp
+++ b/installer/src/installer-lib/test/test-installer-lib-sandbox.cpp
@@ -80,9 +80,8 @@
{
session.log( "log_IE_window_handles" ) ;
const wchar_t * IE_names[] = { L"IExplore.exe", L"AdblockPlusEngine.exe" } ;
- const wchar_t * ABP_names[] = { L"AdblockPlus32.dll", L"AdblockPlus64.dll" } ;
Process_Snapshot snapshot ;
- Process_Closer iec( snapshot, IE_names, ABP_names) ;
+ Process_Closer iec(snapshot, IE_names) ;
log_single_window_handle_only_if_IE lp( session, iec ) ;
enumerate_windows( lp ) ;
}
@@ -94,9 +93,8 @@
{
session.log( "log_only_window_handle_in_closer" ) ;
const wchar_t * IE_names[] = { L"IExplore.exe", L"AdblockPlusEngine.exe" } ;
- const wchar_t * ABP_names[] = { L"AdblockPlus32.dll", L"AdblockPlus64.dll" } ;
Process_Snapshot snapshot ;
- Process_Closer iec( snapshot, IE_names, ABP_names) ;
+ Process_Closer iec( snapshot, IE_names) ;
iec.iterate_our_windows( log_single_window_handle( session ) ) ;
}

Powered by Google App Engine
This is Rietveld