change overlay scrollbar hover show to hover fade in

cc/input/scrollbar_animation_controller.cc

 // Copyright 2014 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 "cc/input/scrollbar_animation_controller.h"
 
 #include <algorithm>
 
 #include "base/time/time.h"
 #include "cc/trees/layer_tree_impl.h"
@@ skipping 12 matching lines @@
       fade_out_duration));
 }
 
 std::unique_ptr<ScrollbarAnimationController>
 ScrollbarAnimationController::CreateScrollbarAnimationControllerAuraOverlay(
     int scroll_layer_id,
     ScrollbarAnimationControllerClient* client,
     base::TimeDelta show_delay,
     base::TimeDelta fade_out_delay,
     base::TimeDelta fade_out_resize_delay,
+    base::TimeDelta fade_in_duration,
     base::TimeDelta fade_out_duration,
     base::TimeDelta thinning_duration) {
   return base::WrapUnique(new ScrollbarAnimationController(
       scroll_layer_id, client, show_delay, fade_out_delay,
-      fade_out_resize_delay, fade_out_duration, thinning_duration));
+      fade_out_resize_delay, fade_in_duration, fade_out_duration,
+      thinning_duration));
 }
 
 ScrollbarAnimationController::ScrollbarAnimationController(
     int scroll_layer_id,
     ScrollbarAnimationControllerClient* client,
     base::TimeDelta fade_out_delay,
     base::TimeDelta fade_out_resize_delay,
     base::TimeDelta fade_out_duration)
     : client_(client),
       fade_out_delay_(fade_out_delay),
       fade_out_resize_delay_(fade_out_resize_delay),
+      fade_out_duration_(fade_out_duration),
       need_trigger_scrollbar_show_(false),
       is_animating_(false),
+      animation_change_(NONE),
       scroll_layer_id_(scroll_layer_id),
       currently_scrolling_(false),
       show_in_fast_scroll_(false),
       opacity_(0.0f),
-      fade_out_duration_(fade_out_duration),
       show_scrollbars_on_scroll_gesture_(false),
       need_thinning_animation_(false),
       weak_factory_(this) {
   ApplyOpacityToScrollbars(0.0f);
 }
 
 ScrollbarAnimationController::ScrollbarAnimationController(
     int scroll_layer_id,
     ScrollbarAnimationControllerClient* client,
     base::TimeDelta show_delay,
     base::TimeDelta fade_out_delay,
     base::TimeDelta fade_out_resize_delay,
+    base::TimeDelta fade_in_duration,
     base::TimeDelta fade_out_duration,
     base::TimeDelta thinning_duration)
     : client_(client),
       show_delay_(show_delay),
       fade_out_delay_(fade_out_delay),
       fade_out_resize_delay_(fade_out_resize_delay),
+      fade_in_duration_(fade_in_duration),
+      fade_out_duration_(fade_out_duration),
       need_trigger_scrollbar_show_(false),
       is_animating_(false),
+      animation_change_(NONE),
       scroll_layer_id_(scroll_layer_id),
       currently_scrolling_(false),
       show_in_fast_scroll_(false),
       opacity_(0.0f),
-      fade_out_duration_(fade_out_duration),
       show_scrollbars_on_scroll_gesture_(true),
       need_thinning_animation_(true),
       weak_factory_(this) {
   vertical_controller_ = SingleScrollbarAnimationControllerThinning::Create(
       scroll_layer_id, ScrollbarOrientation::VERTICAL, client,
       thinning_duration);
   horizontal_controller_ = SingleScrollbarAnimationControllerThinning::Create(
       scroll_layer_id, ScrollbarOrientation::HORIZONTAL, client,
       thinning_duration);
   ApplyOpacityToScrollbars(0.0f);
@@ skipping 20 matching lines @@
   delayed_scrollbar_fade_out_.Cancel();
   is_animating_ = true;
   last_awaken_time_ = base::TimeTicks();
   client_->SetNeedsAnimateForScrollbarAnimation();
 }
 
 void ScrollbarAnimationController::StopAnimation() {
   delayed_scrollbar_show_.Cancel();
   delayed_scrollbar_fade_out_.Cancel();
   is_animating_ = false;
+  animation_change_ = NONE;
 }
 
-void ScrollbarAnimationController::PostDelayedShow() {
+void ScrollbarAnimationController::PostDelayedFadeIn() {
   DCHECK(delayed_scrollbar_fade_out_.IsCancelled());
+  animation_change_ = FADE_IN;
   delayed_scrollbar_show_.Cancel();
-  delayed_scrollbar_show_.Reset(base::Bind(&ScrollbarAnimationController::Show,
-                                           weak_factory_.GetWeakPtr()));
+  delayed_scrollbar_show_.Reset(

[bokan, 2017/04/13 14:38:14]

Rename delayed_scrollbar_show too.

[chaopeng, 2017/04/13 15:15:37]

Done.

+      base::Bind(&ScrollbarAnimationController::StartAnimation,
+                 weak_factory_.GetWeakPtr()));
   client_->PostDelayedScrollbarAnimationTask(delayed_scrollbar_show_.callback(),
                                              show_delay_);
 }
 
 void ScrollbarAnimationController::PostDelayedFadeOut(bool on_resize) {
   DCHECK(delayed_scrollbar_show_.IsCancelled());
+  animation_change_ = FADE_OUT;
   base::TimeDelta delay = on_resize ? fade_out_resize_delay_ : fade_out_delay_;
   delayed_scrollbar_fade_out_.Cancel();
   delayed_scrollbar_fade_out_.Reset(
       base::Bind(&ScrollbarAnimationController::StartAnimation,
                  weak_factory_.GetWeakPtr()));
   client_->PostDelayedScrollbarAnimationTask(
       delayed_scrollbar_fade_out_.callback(), delay);
 }
 
 bool ScrollbarAnimationController::Animate(base::TimeTicks now) {
   bool animated = false;
 
-  if (is_animating_) {
+  if (is_animating_ && animation_change_ != NONE) {
     if (last_awaken_time_.is_null())
       last_awaken_time_ = now;
 
     float progress = AnimationProgressAtTime(now);
     RunAnimationFrame(progress);
 
     if (is_animating_)
       client_->SetNeedsAnimateForScrollbarAnimation();
     animated = true;
   }
 
   if (need_thinning_animation_) {
     animated |= vertical_controller_->Animate(now);
     animated |= horizontal_controller_->Animate(now);
   }
 
   return animated;
 }
 
 float ScrollbarAnimationController::AnimationProgressAtTime(
     base::TimeTicks now) {
   base::TimeDelta delta = now - last_awaken_time_;
-  float progress = delta.InSecondsF() / fade_out_duration_.InSecondsF();
+  base::TimeDelta duration =
+      animation_change_ == FADE_IN ? fade_in_duration_ : fade_out_duration_;
+  float progress = delta.InSecondsF() / duration.InSecondsF();
   return std::max(std::min(progress, 1.f), 0.f);
 }
 
 void ScrollbarAnimationController::RunAnimationFrame(float progress) {
-  ApplyOpacityToScrollbars(1.f - progress);
+  float opacity;
+
+  if (animation_change_ == FADE_IN) {
+    opacity = std::max(progress, opacity_);
+  } else {

[bokan, 2017/04/13 14:38:14]

DCHECK above that animation_change_ isn't NONE

[chaopeng, 2017/04/13 15:15:37]

Done.

+    opacity = std::min(1.f - progress, opacity_);
+  }
+
+  ApplyOpacityToScrollbars(opacity);
   if (progress == 1.f)
     StopAnimation();
 }
 
 void ScrollbarAnimationController::DidScrollBegin() {
   currently_scrolling_ = true;
 }
 
 void ScrollbarAnimationController::DidScrollEnd() {
   bool has_scrolled = show_in_fast_scroll_;
@@ skipping 104 matching lines @@
 
   need_trigger_scrollbar_show_ =
       CalcNeedTriggerScrollbarShow(orientation, distance);
 
   if (Captured())
     return;
 
   if (ScrollbarsHidden()) {
     if (need_trigger_scrollbar_show_before != need_trigger_scrollbar_show_) {
       if (need_trigger_scrollbar_show_) {
-        PostDelayedShow();
+        PostDelayedFadeIn();
       } else {
         delayed_scrollbar_show_.Cancel();
       }
     }
   } else {
     if (MouseIsNearAnyScrollbar()) {
       Show();
       StopAnimation();
     } else if (!is_animating_) {
       PostDelayedFadeOut(false);
     }
   }
 }
 
 bool ScrollbarAnimationController::CalcNeedTriggerScrollbarShow(
     ScrollbarOrientation orientation,
     float distance) const {
   DCHECK(need_thinning_animation_);
 
   if (vertical_controller_->mouse_is_over_scrollbar() ||
       horizontal_controller_->mouse_is_over_scrollbar())
     return true;
 
   for (ScrollbarLayerImplBase* scrollbar : Scrollbars()) {
     if (scrollbar->orientation() != orientation)
       continue;
 
-    if (distance < kMouseMoveDistanceToTriggerShow)
+    if (distance < kMouseMoveDistanceToTriggerFadeIn)
       return true;
   }
 
   return false;
 }
 
 bool ScrollbarAnimationController::MouseIsOverScrollbar(
     ScrollbarOrientation orientation) const {
   DCHECK(need_thinning_animation_);
   return GetScrollbarAnimationController(orientation).mouse_is_over_scrollbar();
@@ skipping 14 matching lines @@
 bool ScrollbarAnimationController::ScrollbarsHidden() const {
   return opacity_ == 0.0f;
 }
 
 bool ScrollbarAnimationController::Captured() const {
   DCHECK(need_thinning_animation_);
   return vertical_controller_->captured() || horizontal_controller_->captured();
 }
 
 void ScrollbarAnimationController::Show() {
-  delayed_scrollbar_show_.Cancel();

[bokan, 2017/04/13 14:38:14]

Why don't we want to cancel this now? If we scroll while the mouse is nearby,
this will cause the scrollbars to show instantly - at that point, we don't need
the "fade in due to hover" animation anymore, right?

[chaopeng, 2017/04/13 15:15:37]

Yes, we should keep this.

   ApplyOpacityToScrollbars(1.0f);
 }
 
 void ScrollbarAnimationController::ApplyOpacityToScrollbars(float opacity) {
   for (ScrollbarLayerImplBase* scrollbar : Scrollbars()) {
     if (!scrollbar->is_overlay_scrollbar())
       continue;
     float effective_opacity = scrollbar->CanScrollOrientation() ? opacity : 0;
     scrollbar->SetOverlayScrollbarLayerOpacityAnimated(effective_opacity);
   }
 
   bool previouslyVisible = opacity_ > 0.0f;
   bool currentlyVisible = opacity > 0.0f;
 
   if (opacity_ != opacity)
     client_->SetNeedsRedrawForScrollbarAnimation();
 
   opacity_ = opacity;
 
   if (previouslyVisible != currentlyVisible)
     client_->DidChangeScrollbarVisibility();
 }
 
 }  // namespace cc