Chromium Code Reviews| Index: chrome/browser/ui/toolbar/app_menu_icon_controller.h |
| diff --git a/chrome/browser/ui/toolbar/app_menu_badge_controller.h b/chrome/browser/ui/toolbar/app_menu_icon_controller.h |
| similarity index 74% |
| rename from chrome/browser/ui/toolbar/app_menu_badge_controller.h |
| rename to chrome/browser/ui/toolbar/app_menu_icon_controller.h |
| index 76ec6c7bd25dd4ca5e1ec352662f8b1400490a76..d4f80a80141704ac3e058c91f2e0665fe6e1bbc7 100644 |
| --- a/chrome/browser/ui/toolbar/app_menu_badge_controller.h |
| +++ b/chrome/browser/ui/toolbar/app_menu_icon_controller.h |
| @@ -2,8 +2,8 @@ |
| // Use of this source code is governed by a BSD-style license that can be |
| // found in the LICENSE file. |
| -#ifndef CHROME_BROWSER_UI_TOOLBAR_APP_MENU_BADGE_CONTROLLER_H_ |
| -#define CHROME_BROWSER_UI_TOOLBAR_APP_MENU_BADGE_CONTROLLER_H_ |
| +#ifndef CHROME_BROWSER_UI_TOOLBAR_APP_MENU_ICON_CONTROLLER_H_ |
| +#define CHROME_BROWSER_UI_TOOLBAR_APP_MENU_ICON_CONTROLLER_H_ |
| #include "base/macros.h" |
| #include "chrome/browser/ui/toolbar/app_menu_icon_painter.h" |
| @@ -16,9 +16,9 @@ class Profile; |
| // AppMenuBadgeController encapsulates the logic for badging the app menu icon |
|
msw
2016/06/07 01:01:38
nit: update all mentions of badge in comments, ple
|
| // as a result of various events - such as available updates, errors, etc. |
| -class AppMenuBadgeController : public content::NotificationObserver { |
| +class AppMenuIconController : public content::NotificationObserver { |
| public: |
| - enum class BadgeType { |
| + enum class IconType { |
| NONE, |
| UPGRADE_NOTIFICATION, |
| GLOBAL_ERROR, |
| @@ -31,9 +31,9 @@ class AppMenuBadgeController : public content::NotificationObserver { |
| // Notifies the UI to update the badge to have the specified |severity|, as |
| // well as specifying whether it should |animate|. The |type| parameter |
| // specifies the type of change (i.e. the source of the notification). |
| - virtual void UpdateBadgeSeverity(BadgeType type, |
| - AppMenuIconPainter::Severity severity, |
| - bool animate) = 0; |
| + virtual void UpdateIconSeverity(IconType type, |
| + AppMenuIconPainter::Severity severity, |
| + bool animate) = 0; |
| protected: |
| virtual ~Delegate() {} |
| @@ -41,8 +41,8 @@ class AppMenuBadgeController : public content::NotificationObserver { |
| // Creates an instance of this class for the given |profile| that will notify |
| // |delegate| of updates. |
| - AppMenuBadgeController(Profile* profile, Delegate* delegate); |
| - ~AppMenuBadgeController() override; |
| + AppMenuIconController(Profile* profile, Delegate* delegate); |
| + ~AppMenuIconController() override; |
| // Forces an update of the UI based on the current state of the world. This |
| // will check whether there are any current pending updates, global errors, |
| @@ -60,7 +60,7 @@ class AppMenuBadgeController : public content::NotificationObserver { |
| Delegate* delegate_; |
| content::NotificationRegistrar registrar_; |
| - DISALLOW_COPY_AND_ASSIGN(AppMenuBadgeController); |
| + DISALLOW_COPY_AND_ASSIGN(AppMenuIconController); |
| }; |
| -#endif // CHROME_BROWSER_UI_TOOLBAR_APP_MENU_BADGE_CONTROLLER_H_ |
| +#endif // CHROME_BROWSER_UI_TOOLBAR_APP_MENU_ICON_CONTROLLER_H_ |