Index: chrome/browser/extensions/extension_prefs_unittest.cc |
diff --git a/chrome/browser/extensions/extension_prefs_unittest.cc b/chrome/browser/extensions/extension_prefs_unittest.cc |
index 22a8c50ca3bf652230dd8f1adbc71308a241c046..67eb2a1fa16b8bcf7a6447d97c5d6aae7cb94c52 100644 |
--- a/chrome/browser/extensions/extension_prefs_unittest.cc |
+++ b/chrome/browser/extensions/extension_prefs_unittest.cc |
@@ -15,6 +15,7 @@ |
#include "chrome/browser/extensions/extension_pref_value_map.h" |
#include "chrome/browser/extensions/extension_prefs.h" |
#include "chrome/browser/prefs/scoped_user_pref_update.h" |
+#include "chrome/browser/prefs/pref_observer_mock.h" |
#include "chrome/common/chrome_paths.h" |
#include "chrome/common/extensions/extension_manifest_constants.h" |
#include "chrome/common/extensions/permissions/permission_set.h" |
@@ -1052,60 +1053,60 @@ class ExtensionPrefsNotifyWhenNeeded : public ExtensionPrefsPrepopulatedTest { |
using testing::Mock; |
using testing::StrEq; |
- content::MockNotificationObserver observer; |
+ PrefObserverMock observer; |
PrefChangeRegistrar registrar; |
registrar.Init(prefs()->pref_service()); |
registrar.Add(kPref1, &observer); |
- content::MockNotificationObserver incognito_observer; |
+ PrefObserverMock incognito_observer; |
scoped_ptr<PrefService> incog_prefs(prefs_.CreateIncognitoPrefService()); |
PrefChangeRegistrar incognito_registrar; |
incognito_registrar.Init(incog_prefs.get()); |
incognito_registrar.Add(kPref1, &incognito_observer); |
// Write value and check notification. |
- EXPECT_CALL(observer, Observe(_, _, _)); |
- EXPECT_CALL(incognito_observer, Observe(_, _, _)); |
+ EXPECT_CALL(observer, OnPreferenceChanged(_, _)); |
+ EXPECT_CALL(incognito_observer, OnPreferenceChanged(_, _)); |
InstallExtControlledPref(ext1_, kPref1, |
Value::CreateStringValue("https://www.chromium.org")); |
Mock::VerifyAndClearExpectations(&observer); |
Mock::VerifyAndClearExpectations(&incognito_observer); |
// Write same value. |
- EXPECT_CALL(observer, Observe(_, _, _)).Times(0); |
- EXPECT_CALL(incognito_observer, Observe(_, _, _)).Times(0); |
+ EXPECT_CALL(observer, OnPreferenceChanged(_, _)).Times(0); |
+ EXPECT_CALL(incognito_observer, OnPreferenceChanged(_, _)).Times(0); |
InstallExtControlledPref(ext1_, kPref1, |
Value::CreateStringValue("https://www.chromium.org")); |
Mock::VerifyAndClearExpectations(&observer); |
Mock::VerifyAndClearExpectations(&incognito_observer); |
// Change value. |
- EXPECT_CALL(observer, Observe(_, _, _)); |
- EXPECT_CALL(incognito_observer, Observe(_, _, _)); |
+ EXPECT_CALL(observer, OnPreferenceChanged(_, _)); |
+ EXPECT_CALL(incognito_observer, OnPreferenceChanged(_, _)); |
InstallExtControlledPref(ext1_, kPref1, |
Value::CreateStringValue("chrome://newtab")); |
Mock::VerifyAndClearExpectations(&observer); |
Mock::VerifyAndClearExpectations(&incognito_observer); |
// Change only incognito persistent value. |
- EXPECT_CALL(observer, Observe(_, _, _)).Times(0); |
- EXPECT_CALL(incognito_observer, Observe(_, _, _)); |
+ EXPECT_CALL(observer, OnPreferenceChanged(_, _)).Times(0); |
+ EXPECT_CALL(incognito_observer, OnPreferenceChanged(_, _)); |
InstallExtControlledPrefIncognito(ext1_, kPref1, |
Value::CreateStringValue("chrome://newtab2")); |
Mock::VerifyAndClearExpectations(&observer); |
Mock::VerifyAndClearExpectations(&incognito_observer); |
// Change only incognito session-only value. |
- EXPECT_CALL(observer, Observe(_, _, _)).Times(0); |
- EXPECT_CALL(incognito_observer, Observe(_, _, _)); |
+ EXPECT_CALL(observer, OnPreferenceChanged(_, _)).Times(0); |
+ EXPECT_CALL(incognito_observer, OnPreferenceChanged(_, _)); |
InstallExtControlledPrefIncognito(ext1_, kPref1, |
Value::CreateStringValue("chrome://newtab3")); |
Mock::VerifyAndClearExpectations(&observer); |
Mock::VerifyAndClearExpectations(&incognito_observer); |
// Uninstall. |
- EXPECT_CALL(observer, Observe(_, _, _)); |
- EXPECT_CALL(incognito_observer, Observe(_, _, _)); |
+ EXPECT_CALL(observer, OnPreferenceChanged(_, _)); |
+ EXPECT_CALL(incognito_observer, OnPreferenceChanged(_, _)); |
UninstallExtension(ext1_->id()); |
Mock::VerifyAndClearExpectations(&observer); |
Mock::VerifyAndClearExpectations(&incognito_observer); |