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

Unified Diff: chrome/browser/chromeos/input_method/browser_state_monitor_unittest.cc

Issue 11345008: Remove content::NotificationObserver dependency from most Prefs code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix PrefNotifierImpl Created 8 years, 2 months 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/chromeos/input_method/browser_state_monitor_unittest.cc
diff --git a/chrome/browser/chromeos/input_method/browser_state_monitor_unittest.cc b/chrome/browser/chromeos/input_method/browser_state_monitor_unittest.cc
index 3746b803d7e17b77469b22f3e84c512fb9b82e5b..b73b05aa4bb8946e1392cf55956ce3a89767aa24 100644
--- a/chrome/browser/chromeos/input_method/browser_state_monitor_unittest.cc
+++ b/chrome/browser/chromeos/input_method/browser_state_monitor_unittest.cc
@@ -237,22 +237,6 @@ TEST(BrowserStateMonitorTest, TestObserveAppTerminating) {
EXPECT_EQ(0, monitor.update_user_pref_count());
}
-TEST(BrowserStateMonitorTest, TestObservePrefChanged) {
- TestingPrefService prefs;
- RegisterTestPrefs(&prefs);
- MockInputMethodManager mock_manager;
- TestableBrowserStateMonitor monitor(&mock_manager);
- monitor.SetPrefServiceForTesting(&prefs);
-
- EXPECT_EQ(1, mock_manager.set_state_count_);
- monitor.Observe(chrome::NOTIFICATION_PREF_CHANGED,
- content::NotificationService::AllSources(),
- content::NotificationService::NoDetails());
-
- // NOTIFICATION_PREF_CHANGED should be ignored.
- EXPECT_EQ(1, mock_manager.set_state_count_);
-}
-
TEST(BrowserStateMonitorTest, TestUpdatePrefOnLoginScreen) {
TestingPrefService prefs;
RegisterTestPrefs(&prefs);

Powered by Google App Engine
This is Rietveld 408576698