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

Unified Diff: chrome/browser/ui/views/toolbar/app_menu_button.h

Issue 2843413003: [Views] App Menu Icon Update (Closed)
Patch Set: Fix for isherman Created 3 years, 8 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/views/toolbar/app_menu_button.h
diff --git a/chrome/browser/ui/views/toolbar/app_menu_button.h b/chrome/browser/ui/views/toolbar/app_menu_button.h
index 06167a79a0e81e727016bb0d51ce661221d4493b..96290b40b659500fc84272b286ff4ab28b5704b1 100644
--- a/chrome/browser/ui/views/toolbar/app_menu_button.h
+++ b/chrome/browser/ui/views/toolbar/app_menu_button.h
@@ -72,6 +72,10 @@ class AppMenuButton : public views::MenuButton, public TabStripModelObserver {
// to make the focus rectangle centered.
void SetTrailingMargin(int margin);
+ // Animates the icon if possible. The icon will not animate if the severity
+ // level is none, |animation_| is nullptr or |should_use_new_icon_| is false.
+ void AnimateIconIfPossible();
+
// Methods called by AppMenuAnimation when the animation has started/ended.
// The layer is managed inside these methods.
void AppMenuAnimationStarted();
@@ -115,6 +119,9 @@ class AppMenuButton : public views::MenuButton, public TabStripModelObserver {
// Used for animating and drawing the app menu icon.
std::unique_ptr<AppMenuAnimation> animation_;
+ // True if the app menu should use the new animated icon.
+ bool should_use_new_icon_;
+
// Any trailing margin to be applied. Used when the browser is in
// a maximized state to extend to the full window width.
int margin_trailing_;

Powered by Google App Engine
This is Rietveld 408576698