[Views] App Menu Icon Update

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

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_TOOLBAR_APP_MENU_ANIMATION_H_
 #define CHROME_BROWSER_UI_VIEWS_TOOLBAR_APP_MENU_ANIMATION_H_
 
 #include "base/time/time.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/gfx/animation/animation_delegate.h"
 #include "ui/gfx/animation/slide_animation.h"
 
 namespace gfx {
 class Canvas;
 class PointF;
 }  // namespace gfx
 
 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();
 
   // gfx::AnimationDelegate:
   void AnimationEnded(const gfx::Animation* animation) override;
   void AnimationProgressed(const gfx::Animation* animation) override;
 
  private:
   // This class is used to represent and paint a dot on the app menu.
@@ skipping 21 matching lines @@
     // The percentage of the overall animation duration it takes to animate the
     // width and stroke to their open state.
     const float width_open_interval_;
     const float stroke_open_interval_;
 
     DISALLOW_COPY_AND_ASSIGN(AppMenuDot);
   };
 
   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_;
   AppMenuDot middle_dot_;
   AppMenuDot top_dot_;
 
   // The starting color of the dots. The animation is expected to transition
   // from this color to |severity_color_|.
   SkColor start_color_;
 
   // The severity color of the dots. This is final color at the end of the
   // animation.
   SkColor severity_color_;
 
   DISALLOW_COPY_AND_ASSIGN(AppMenuAnimation);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TOOLBAR_APP_MENU_ANIMATION_H_