Chromium Code Reviews| Index: chrome/browser/ui/webui/ntp/app_launcher_handler.h |
| diff --git a/chrome/browser/ui/webui/ntp/app_launcher_handler.h b/chrome/browser/ui/webui/ntp/app_launcher_handler.h |
| index 92809303aaa8bdbaa5d922ebff27cbb220d8ca7c..a6e9c582317c0160a2c17d33ee828cac2dddf88f 100644 |
| --- a/chrome/browser/ui/webui/ntp/app_launcher_handler.h |
| +++ b/chrome/browser/ui/webui/ntp/app_launcher_handler.h |
| @@ -9,7 +9,6 @@ |
| #include "base/memory/scoped_ptr.h" |
| #include "base/prefs/public/pref_change_registrar.h" |
| -#include "base/prefs/public/pref_observer.h" |
| #include "chrome/browser/common/cancelable_request.h" |
| #include "chrome/browser/extensions/extension_install_prompt.h" |
| #include "chrome/browser/extensions/extension_uninstall_dialog.h" |
| @@ -33,8 +32,7 @@ class AppNotification; |
| class AppLauncherHandler : public content::WebUIMessageHandler, |
| public ExtensionUninstallDialog::Delegate, |
| public ExtensionInstallPrompt::Delegate, |
| - public content::NotificationObserver, |
| - public PrefObserver { |
| + public content::NotificationObserver { |
| public: |
| explicit AppLauncherHandler(ExtensionService* extension_service); |
| virtual ~AppLauncherHandler(); |
| @@ -54,10 +52,6 @@ class AppLauncherHandler : public content::WebUIMessageHandler, |
| const content::NotificationSource& source, |
| const content::NotificationDetails& details) OVERRIDE; |
| - // PrefObserver |
| - virtual void OnPreferenceChanged(PrefServiceBase* service, |
| - const std::string& pref_name) OVERRIDE; |
| - |
| // Populate the given dictionary with all installed app info. |
| void FillAppDictionary(base::DictionaryValue* value); |
| @@ -166,6 +160,8 @@ class AppLauncherHandler : public content::WebUIMessageHandler, |
| // Sends |highlight_app_id_| to the js. |
| void SetAppToBeHighlighted(); |
| + void OnPreferencesChanged(); |
|
Jói
2012/11/20 22:11:55
suggest OnPreferenceChanged (singular) to match na
tfarina
2012/11/20 23:41:20
Done.
|
| + |
| // The apps are represented in the extensions model, which |
| // outlives us since it's owned by our containing profile. |
| ExtensionService* const extension_service_; |