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

Unified Diff: chrome/browser/ui/toolbar/app_menu_icon_controller.h

Issue 2925973002: Migrate upgrade notifications to observer (Closed)
Patch Set: Fix test failure Created 3 years, 6 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/ui/toolbar/app_menu_icon_controller.h
diff --git a/chrome/browser/ui/toolbar/app_menu_icon_controller.h b/chrome/browser/ui/toolbar/app_menu_icon_controller.h
index 81f6a759101430fa9b28754b8eec2eb7c134425d..2e7a98e5617eaa606476f9bb6bf7e56f74ea541e 100644
--- a/chrome/browser/ui/toolbar/app_menu_icon_controller.h
+++ b/chrome/browser/ui/toolbar/app_menu_icon_controller.h
@@ -6,6 +6,7 @@
#define CHROME_BROWSER_UI_TOOLBAR_APP_MENU_ICON_CONTROLLER_H_
#include "base/macros.h"
+#include "chrome/browser/upgrade_observer.h"
#include "content/public/browser/notification_details.h"
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
@@ -23,7 +24,8 @@ class AppMenuIconController :
#if defined(OS_WIN)
public EnumerateModulesModel::Observer,
#endif
- public content::NotificationObserver {
+ public content::NotificationObserver,
+ public UpgradeObserver {
public:
enum class IconType {
NONE,
@@ -69,6 +71,9 @@ class AppMenuIconController :
const content::NotificationSource& source,
const content::NotificationDetails& details) override;
+ // UpgradeObserver implementation.
+ void OnUpgradeRecommended() override;
+
#if defined(OS_WIN)
// EnumerateModulesModel:
void OnScanCompleted() override;

Powered by Google App Engine
This is Rietveld 408576698