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

Unified Diff: chrome/browser/prefs/pref_change_registrar_unittest.cc

Issue 5646003: Sanitize PrefStore interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase, fix up unit tests. Created 10 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/browser/prefs/pref_change_registrar_unittest.cc
diff --git a/chrome/browser/prefs/pref_change_registrar_unittest.cc b/chrome/browser/prefs/pref_change_registrar_unittest.cc
index 33101671985155b007a85b8c51345df0909c6a8f..2e4cd0e8425ef38c00ba824fddfb7a570508901a 100644
--- a/chrome/browser/prefs/pref_change_registrar_unittest.cc
+++ b/chrome/browser/prefs/pref_change_registrar_unittest.cc
@@ -15,6 +15,8 @@
using testing::Mock;
using testing::Eq;
+namespace {
+
// A mock provider that allows us to capture pref observer changes.
class MockPrefService : public TestingPrefService {
danno 2010/12/08 13:08:45 Choice of mock name if you clean up the other Mock
Mattias Nissler (ping if slow) 2010/12/09 10:20:20 Doesn't apply any longer.
public:
@@ -25,6 +27,8 @@ class MockPrefService : public TestingPrefService {
MOCK_METHOD2(RemovePrefObserver, void(const char*, NotificationObserver*));
};
+} // namespace
+
class PrefChangeRegistrarTest : public testing::Test {
public:
PrefChangeRegistrarTest() {}

Powered by Google App Engine
This is Rietveld 408576698