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

Unified Diff: chrome/browser/ui/webui/options/browser_options_handler.cc

Issue 11345008: Remove content::NotificationObserver dependency from most Prefs code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge to head for commit Created 8 years, 1 month 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/ui/webui/options/browser_options_handler.cc
diff --git a/chrome/browser/ui/webui/options/browser_options_handler.cc b/chrome/browser/ui/webui/options/browser_options_handler.cc
index 8275604356ac3a527ae26a117b19f09887765417..47d6e35d35304c6cadfd6faad6d943849fb7659e 100644
--- a/chrome/browser/ui/webui/options/browser_options_handler.cc
+++ b/chrome/browser/ui/webui/options/browser_options_handler.cc
@@ -866,32 +866,34 @@ void BrowserOptionsHandler::Observe(
} else if (type == chrome::NOTIFICATION_LOGIN_USER_IMAGE_CHANGED) {
UpdateAccountPicture();
#endif
- } else if (type == chrome::NOTIFICATION_PREF_CHANGED) {
- std::string* pref_name = content::Details<std::string>(details).ptr();
- if (*pref_name == prefs::kDefaultBrowserSettingEnabled) {
- UpdateDefaultBrowserState();
- } else if (*pref_name == prefs::kDownloadExtensionsToOpen) {
- SetupAutoOpenFileTypes();
+ } else if (type == chrome::NOTIFICATION_PROFILE_CACHED_INFO_CHANGED) {
+ if (multiprofile_)
+ SendProfilesInfo();
+ } else {
+ NOTREACHED();
+ }
+}
+
+void BrowserOptionsHandler::OnPreferenceChanged(PrefServiceBase* service,
+ const std::string& pref_name) {
+ if (pref_name == prefs::kDefaultBrowserSettingEnabled) {
+ UpdateDefaultBrowserState();
+ } else if (pref_name == prefs::kDownloadExtensionsToOpen) {
+ SetupAutoOpenFileTypes();
#if !defined(OS_CHROMEOS)
- } else if (proxy_prefs_.IsObserved(*pref_name)) {
- SetupProxySettingsSection();
+ } else if (proxy_prefs_.IsObserved(pref_name)) {
+ SetupProxySettingsSection();
#endif // !defined(OS_CHROMEOS)
- } else if ((*pref_name == prefs::kCloudPrintEmail) ||
- (*pref_name == prefs::kCloudPrintProxyEnabled)) {
+ } else if ((pref_name == prefs::kCloudPrintEmail) ||
+ (pref_name == prefs::kCloudPrintProxyEnabled)) {
#if !defined(OS_CHROMEOS)
- if (cloud_print_connector_ui_enabled_)
- SetupCloudPrintConnectorSection();
+ if (cloud_print_connector_ui_enabled_)
+ SetupCloudPrintConnectorSection();
#endif
- } else if (*pref_name == prefs::kWebKitDefaultFontSize) {
- SetupFontSizeSelector();
- } else if (*pref_name == prefs::kDefaultZoomLevel) {
- SetupPageZoomSelector();
- } else {
- NOTREACHED();
- }
- } else if (type == chrome::NOTIFICATION_PROFILE_CACHED_INFO_CHANGED) {
- if (multiprofile_)
- SendProfilesInfo();
+ } else if (pref_name == prefs::kWebKitDefaultFontSize) {
+ SetupFontSizeSelector();
+ } else if (pref_name == prefs::kDefaultZoomLevel) {
+ SetupPageZoomSelector();
} else {
NOTREACHED();
}

Powered by Google App Engine
This is Rietveld 408576698