[Views] App Menu Icon Update

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

 // 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.
 
 #include "chrome/browser/ui/views/toolbar/app_menu_animation.h"
 
 #include "base/memory/ptr_util.h"
 #include "cc/paint/paint_flags.h"
 #include "chrome/browser/ui/views/toolbar/app_menu_button.h"
 #include "ui/gfx/animation/tween.h"
@@ skipping 106 matching lines @@
   flags.setColor(color);
   flags.setStrokeWidth(bounds.height() * kCloseStroke);
   flags.setStrokeCap(cc::PaintFlags::kRound_Cap);
   flags.setStyle(cc::PaintFlags::kFill_Style);
   flags.setAntiAlias(true);
 
   gfx::SizeF dot_size = gfx::SizeF(dot_width, dot_height);
   canvas->DrawRoundRect(gfx::RectF(point, dot_size), 2.0, flags);
 }
 
-AppMenuAnimation::AppMenuAnimation(AppMenuButton* owner,
-                                   bool should_animate_closed)
+AppMenuAnimation::AppMenuAnimation(AppMenuButton* owner, SkColor initial_color)
     : owner_(owner),
-      should_animate_closed_(should_animate_closed),
       animation_(base::MakeUnique<gfx::SlideAnimation>(this)),
       bottom_dot_(base::TimeDelta(),
                   kBottomWidthOpenInterval,
                   kBottomStrokeOpenInterval),
       middle_dot_(base::TimeDelta::FromMilliseconds(kDotDelayMs),
                   kMiddleWidthOpenInterval,
                   kMiddleStrokeOpenInterval),
       top_dot_(base::TimeDelta::FromMilliseconds(kDotDelayMs * 2),
                kTopWidthOpenInterval,
                kTopStrokeOpenInterval),
-      start_color_(gfx::kPlaceholderColor),
-      severity_color_(gfx::kPlaceholderColor) {
+      start_color_(initial_color),
+      severity_color_(initial_color) {
   animation_->SetSlideDuration(kOpenDurationMs);
   animation_->SetTweenType(gfx::Tween::FAST_OUT_SLOW_IN);
 }
 
 AppMenuAnimation::~AppMenuAnimation() {}
 
 void AppMenuAnimation::PaintAppMenu(gfx::Canvas* canvas,
                                     const gfx::Rect& bounds) {
   gfx::PointF middle_point = gfx::PointF(bounds.CenterPoint());
   float y_offset = kDotYOffset * bounds.height();
   gfx::PointF top_point = middle_point;
   top_point.Offset(0, -y_offset);
 
   gfx::PointF bottom_point = middle_point;
   bottom_point.Offset(0, y_offset);
 
   middle_dot_.Paint(middle_point, start_color_, severity_color_, canvas, bounds,
                     animation_.get());
   top_dot_.Paint(top_point, start_color_, severity_color_, canvas, bounds,
                  animation_.get());
   bottom_dot_.Paint(bottom_point, start_color_, severity_color_, canvas, bounds,
                     animation_.get());
 }
 
-void AppMenuAnimation::SetIconColors(SkColor start_color,
-                                     SkColor severity_color) {
-  start_color_ = start_color;
-  severity_color_ = severity_color;
-}
-
 void AppMenuAnimation::StartAnimation() {
   if (!animation_->is_animating()) {
     animation_->SetSlideDuration(kOpenDurationMs);
     animation_->Show();
     owner_->AppMenuAnimationStarted();
   }
 }
 
 void AppMenuAnimation::AnimationEnded(const gfx::Animation* animation) {
-  if (animation_->IsShowing() && should_animate_closed_) {
+  if (animation_->IsShowing()) {
     animation_->SetSlideDuration(kCloseDurationMs);
     animation_->Hide();
-    return;
+  } else {
+    start_color_ = severity_color_;
   }
 
-  if (!animation_->IsShowing())
-    start_color_ = severity_color_;
-
   owner_->AppMenuAnimationEnded();
 }
 
 void AppMenuAnimation::AnimationProgressed(const gfx::Animation* animation) {
   owner_->SchedulePaint();
 }