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

Unified Diff: installer/src/custom-action/close_application.cpp

Issue 6003395731128320: Only take into account processes that have our plugin loaded (Closed)
Patch Set: Simplify Process_Closer constructor Created March 31, 2014, 8:31 a.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
« no previous file with comments | « installer/installer.gyp ('k') | installer/src/installer-lib/process.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: installer/src/custom-action/close_application.cpp
===================================================================
--- a/installer/src/custom-action/close_application.cpp
+++ b/installer/src/custom-action/close_application.cpp
@@ -11,6 +11,11 @@
#include "interaction.h"
#include "custom-i18n.h"
+const wchar_t * ie_names[] = { L"IExplore.exe" } ;
+const wchar_t * engine_names[] = { L"AdblockPlusEngine.exe" } ;
+const wchar_t * abp_module_names[] = { L"AdblockPlus32.dll", L"AdblockPlus64.dll" } ;
+
+
//-------------------------------------------------------
//-------------------------------------------------------
class IE_Closer
@@ -21,12 +26,9 @@
Process_Closer engine_closer ;
- static const wchar_t * ie_names[] ;
- static const wchar_t * engine_names[] ;
-
public:
IE_Closer()
- : snapshot(), ie_closer( snapshot, ie_names, 1 ), engine_closer( snapshot, engine_names, 1 )
+ : snapshot(), ie_closer( snapshot, ie_names, abp_module_names), engine_closer( snapshot, engine_names )
{}
void refresh()
@@ -59,8 +61,6 @@
}
} ;
-const wchar_t * IE_Closer::ie_names[] = { L"IExplore.exe" } ;
-const wchar_t * IE_Closer::engine_names[] = { L"AdblockPlusEngine.exe" } ;
//-------------------------------------------------------
// abp_close_ie
« no previous file with comments | « installer/installer.gyp ('k') | installer/src/installer-lib/process.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld