Merge Compositor's ScrollbarAnimationControllers into single class |
|||||
Issue descriptionCurrently, there exist two descendants of the ScrollbarAnimationController class in CC: ScrollbarAnimationControllerLinearFade for Android scrollbars and ScrollbarAnimationControllerThinning for Aura overlay scrollbars. The two have similar duties, the difference being the timing constants and the addition of a "thinning/expanding" animation in the latter for when the mouse moves near the scrollbars. Since so much is shared, it would make more sense to merge the two classes and make the behavior differences parameters.
,
Dec 9 2016
,
Feb 8 2017
,
Feb 14 2017
,
Feb 18 2017
Code Review: https://bugs.chromium.org/p/chromium/issues/detail?id=656606
,
Feb 21 2017
Presumably you meant: https://codereview.chromium.org/2692243005/ :) In any case, not sure why the bot didn't paste the commit here. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by dtapu...@chromium.org
, Oct 17 2016