[Views] App Menu Icon Update

chrome/browser/ui/views/toolbar/app_menu_animation.h

Index: chrome/browser/ui/views/toolbar/app_menu_animation.h
diff --git a/chrome/browser/ui/views/toolbar/app_menu_animation.h b/chrome/browser/ui/views/toolbar/app_menu_animation.h
index 1ff69ee156621f0d50fbe03cf856f9c46ddb6afc..5610854c1835c87109f7728945808ed2a136816c 100644
--- a/chrome/browser/ui/views/toolbar/app_menu_animation.h
+++ b/chrome/browser/ui/views/toolbar/app_menu_animation.h
@@ -20,15 +20,16 @@ class AppMenuButton;
 // This class is used for animating and drawing the app menu icon.
 class AppMenuAnimation : public gfx::AnimationDelegate {
  public:
-  AppMenuAnimation(AppMenuButton* owner, bool should_animate_closed);
+  AppMenuAnimation(AppMenuButton* owner, SkColor initial_color);
 
   ~AppMenuAnimation() override;
 
   // Paints the app menu icon.
   void PaintAppMenu(gfx::Canvas* canvas, const gfx::Rect& bounds);
 
-  // Updates the icon colors.
-  void SetIconColors(SkColor start_color, SkColor severity_color);
+  void set_severity_color(SkColor severity_color) {

[msw, 2017/04/29 00:55:05]

nit: maybe rename this and the member to |target_color|?

[spqchan, 2017/04/29 02:34:42]

Done.

+    severity_color_ = severity_color;
+  }
 
   // Starts the animation if it's not already running.
   void StartAnimation();
@@ -70,9 +71,6 @@ class AppMenuAnimation : public gfx::AnimationDelegate {
 
   AppMenuButton* const owner_;
 
-  // True if the animation should close after it finishes opening.
-  const bool should_animate_closed_;
-
   std::unique_ptr<gfx::SlideAnimation> animation_;
 
   AppMenuDot bottom_dot_;