Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(590)

Unified Diff: chrome/common/conflicts/module_watcher_win_unittest.cc

Issue 2576843002: [win] Create ModuleDatabase and ModuleEventSinkImpl. (Closed)
Patch Set: Rework OnProcessStarted. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/conflicts/module_watcher_win_unittest.cc
diff --git a/chrome/common/conflicts/module_watcher_win_unittest.cc b/chrome/common/conflicts/module_watcher_win_unittest.cc
index 02bea081bc52566ce5d3c13bc03673e3b2e5e5fc..dd6988746db4da685be5a165063f2456486956ad 100644
--- a/chrome/common/conflicts/module_watcher_win_unittest.cc
+++ b/chrome/common/conflicts/module_watcher_win_unittest.cc
@@ -10,7 +10,7 @@
#include "testing/gtest/include/gtest/gtest.h"
class ModuleWatcherTest : public testing::Test {
- public:
+ protected:
ModuleWatcherTest()
: module_(nullptr),
module_event_count_(0),
@@ -18,7 +18,7 @@ class ModuleWatcherTest : public testing::Test {
module_loaded_event_count_(0),
module_unloaded_event_count_(0) {}
- void OnModuleEvent(const mojom::ModuleEvent& event) {
+ void OnModuleEvent(const ModuleWatcher::ModuleEvent& event) {
++module_event_count_;
switch (event.event_type) {
case mojom::ModuleEventType::MODULE_ALREADY_LOADED:
@@ -63,7 +63,6 @@ class ModuleWatcherTest : public testing::Test {
// Holds a handle to a loaded module.
HMODULE module_;
-
// Total number of module events seen.
int module_event_count_;
// Total number of MODULE_ALREADY_LOADED events seen.

Powered by Google App Engine
This is Rietveld 408576698