New issue
Advanced search Search tips

Issue 690052 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Overlay scrollbar animation incorrect when capture scrollbar and move out of div then mouse up

Project Member Reported by chaopeng@chromium.org, Feb 8 2017

Issue description

Chrome Version: 56.0.2924.76
OS: Linux

What steps will reproduce the problem?
(1) open http://ht.chaopeng.me/overlay-scrollbar-not-exit.html
(2) capture green div scrollbar
(3) move mouse to red div
(4) release mouse

What is the expected result?

green div scrollbar should get thinning and fade out

 
I just notice this today too.
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 13 2017

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

commit 7250e1ab8f60d4a7ad8644c11ed0576f2c4f34b1
Author: chaopeng <chaopeng@chromium.org>
Date: Mon Feb 13 17:14:43 2017

Pass MouseUp to the captured scrollbar animation controller in layer tree.
This issue is caused by we only pass the MouseUp to the scoll layer currently
mouse over. If we capture the scrollbar then move to another scoll layer then
the captured scrollbar will lose the MouseUp.

In this patch we save the captured_scrollbar_animation_controller when mouse
down so that we can call the captured_scrollbar_animation_controller MouseUp.

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

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

[modify] https://crrev.com/7250e1ab8f60d4a7ad8644c11ed0576f2c4f34b1/cc/trees/layer_tree_host_impl.cc
[modify] https://crrev.com/7250e1ab8f60d4a7ad8644c11ed0576f2c4f34b1/cc/trees/layer_tree_host_impl.h
[modify] https://crrev.com/7250e1ab8f60d4a7ad8644c11ed0576f2c4f34b1/cc/trees/layer_tree_host_impl_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment