New issue
Advanced search Search tips

Issue 636438 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: All , Chrome
Pri: 3
Type: Bug

Blocking:
issue 307091



Sign in to add a comment

Scrollbar does not stay expanded if mouse moves off of scrollbar while dragging

Project Member Reported by mgreenwald@google.com, Aug 10 2016

Issue description

(1) Enable chrome://flags/#overlay-scrollbars
(2) Go to nytimes.com and click and drag scrollbar down
(3) While dragging move cursor off of the scrollbar area and continue dragging down
(4) the scrollbar contracts but should stay expanded until release
 
Labels: OS-Chrome OS-All
Project Member

Comment 2 by sheriffbot@chromium.org, Aug 11 2016

Labels: Hotlist-Google

Comment 3 by bokan@chromium.org, Sep 2 2016

Owner: chaopeng@chromium.org
Status: Started (was: Available)

Comment 5 by bokan@chromium.org, Sep 22 2016

Labels: Hotlist-Input-Dev
Project Member

Comment 6 by bugdroid1@chromium.org, Sep 30 2016

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

commit 54c1c281007900d28cd62c2cc1c6c6cfa3f8325e
Author: chaopeng <chaopeng@chromium.org>
Date: Fri Sep 30 21:17:09 2016

Keep expanded if mouse moves off of scrollbar while dragging

For This bug we add a field `captured_scrollbar_layer_id_` to
layer_tree_host_impl to know when a scrollbar is captured(dragging).
In scrollbar_animation_controller_thinning we only update the state but ignore
the animation when the scrollbar is captured.

BUG= 636438 
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

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

[modify] https://crrev.com/54c1c281007900d28cd62c2cc1c6c6cfa3f8325e/cc/input/input_handler.h
[modify] https://crrev.com/54c1c281007900d28cd62c2cc1c6c6cfa3f8325e/cc/input/scrollbar_animation_controller.h
[modify] https://crrev.com/54c1c281007900d28cd62c2cc1c6c6cfa3f8325e/cc/input/scrollbar_animation_controller_thinning.cc
[modify] https://crrev.com/54c1c281007900d28cd62c2cc1c6c6cfa3f8325e/cc/input/scrollbar_animation_controller_thinning.h
[modify] https://crrev.com/54c1c281007900d28cd62c2cc1c6c6cfa3f8325e/cc/input/scrollbar_animation_controller_thinning_unittest.cc
[modify] https://crrev.com/54c1c281007900d28cd62c2cc1c6c6cfa3f8325e/cc/trees/layer_tree_host_impl.cc
[modify] https://crrev.com/54c1c281007900d28cd62c2cc1c6c6cfa3f8325e/cc/trees/layer_tree_host_impl.h
[modify] https://crrev.com/54c1c281007900d28cd62c2cc1c6c6cfa3f8325e/ui/events/blink/input_handler_proxy.cc
[modify] https://crrev.com/54c1c281007900d28cd62c2cc1c6c6cfa3f8325e/ui/events/blink/input_handler_proxy_unittest.cc

Status: Fixed (was: Started)

Comment 8 by dchan@chromium.org, Oct 7 2016

Labels: VerifyIn-55
Status: Verified (was: Fixed)
Verified on ChromeOS 8872.18.0, 55.0.2883.20

Sign in to add a comment