New issue
Advanced search Search tips

Issue 698275 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Chrome
Pri: 2
Type: Bug

Blocked on:
issue 698334



Sign in to add a comment

Overlay scrollbar "appear on hover" doesn't work until mouse move

Project Member Reported by bokan@chromium.org, Mar 3 2017

Issue description

What steps will reproduce the problem?
(1) Enable overlay scrollbars
(2) Visit any scrollable page, wait for scrollbars to fade out
(3) Move mouse to where the scrollbar was and leave it still

What is the expected result?
The scrollbar should fade in after a delay of ~1 second

What happens instead?
The scrollbar does not fade in. Waiting over 1 second and then moving the mouse causes the scrollbar to fade in.

 
Blockedon: 698334
By bisect, I found this patch (e7a058aa7db4c0fc172200fd16c800ec1b94d8e6 Use nine-patch resource for drawing Aura overlay scrollbar thumb.) caused this issue.

Comment 3 by bokan@chromium.org, Mar 6 2017

Cc: chaopeng@chromium.org
Owner: bokan@chromium.org
Huh, interesting. That patch didn't touch the animation controller at all. Ok, thanks for pointing out the patch, I'll take a look. 
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7858bc70062f09e1cd60e9a99dd8e5129d2c0ff2

commit 7858bc70062f09e1cd60e9a99dd8e5129d2c0ff2
Author: bokan <bokan@chromium.org>
Date: Tue Mar 14 19:22:17 2017

Call setNeedsRedraw when scrollbar opacity changes.

This is needed to ensure that the change to scrollbar opacity is actually
shown if there's no other side effects.

BUG= 698275 
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2740493003
Cr-Commit-Position: refs/heads/master@{#456782}

[modify] https://crrev.com/7858bc70062f09e1cd60e9a99dd8e5129d2c0ff2/cc/input/scrollbar_animation_controller.cc
[modify] https://crrev.com/7858bc70062f09e1cd60e9a99dd8e5129d2c0ff2/cc/trees/layer_tree_host_impl.cc
[modify] https://crrev.com/7858bc70062f09e1cd60e9a99dd8e5129d2c0ff2/cc/trees/layer_tree_host_impl_unittest.cc

Status: Fixed (was: Assigned)
Status: Verified (was: Fixed)
9557.0.0, 60.0.3101.0

Sign in to add a comment