Regression : Scroll bar stuck while scrolling the file list in Sources panel.
Reported by
yfulgaon...@etouch.net,
Jun 6 2016
|
|||||
Issue descriptionChrome version : 53.0.2759.0 (Official Build) 5a1a04452d17cdc3c669949ad74a2f9b6e66b97e-refs/heads/master@{#397936} 32/64 bit OS : Windows (7,8,8.1,10), Mac(10.10.5, 10.11.4), Linux(14.04 LTS) What steps will reproduce the problem? 1) Launch chrome and open devtools. 2) Go to 'Sources' panel and hit Ctrl+P to open a new file. (File list is seen) 3) Now try to scroll the list using mouse and observe. Actual : Scroll bar stuck while scrolling the file list in Sources panel. Expected : Scroll bar should not stuck while scrolling the file list. This is a regression issue, broken in 'M-53', below is the Manual Bisect and will soon update other info. Good Build : 53.0.2757.0 Bad Build : 53.0.2759.0
,
Jun 6 2016
Able to reproduce the issue on win8.1 chrome version 53.0.2760.0 Adding RB label as this is a recent regression
,
Jun 7 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f56bda6f7f5201ace0327e8e64ddf41e9f52d2b5 commit f56bda6f7f5201ace0327e8e64ddf41e9f52d2b5 Author: szager <szager@chromium.org> Date: Tue Jun 07 01:35:16 2016 Restore PaintLayerScrollableArea::ScrollbarManager::canDetach behavior. This CL mistakenly disabled the 'detach' behavior for ScrollbarManager: https://codereview.chromium.org/1930183002 ... because it seemed like DelayScrollPositionClampScope would be sufficient to preserve all the necessary state for scrollbars as they were added and removed during the multiple layout passes of flex layout. However, that was premature: even though the scroll positions were preserved, destroying and recreating scrollbars during layout will invalidate the ScrollingCoordinator's mapping from Scrollbar to WebScrollbarLayer. One side effect is that if a scroll event handler forces layout to run during a scrollbar drag, and the Scrollbar/WebScrollbarLayer association is broken, then the compositor will think that the scrollbar drag has ended, even though mouse-up has not yet happened. This CL also sneaks in a tiny optimization: if auto scrollbars are frozen, then childFlexBasSizeRequiresLayout should not return true due to the child having auto scrollbars (since the scrollbars cannot change). BUG= 617498 R=eae@chromium.org,cbiesinger@chromium.org Review-Url: https://codereview.chromium.org/2042923002 Cr-Commit-Position: refs/heads/master@{#398193} [modify] https://crrev.com/f56bda6f7f5201ace0327e8e64ddf41e9f52d2b5/third_party/WebKit/Source/core/layout/LayoutFlexibleBox.cpp [modify] https://crrev.com/f56bda6f7f5201ace0327e8e64ddf41e9f52d2b5/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp
,
Jun 9 2016
Retested the above issue on All-OS(Windows 7 & 10, Mac 10.11.5 & Ubuntu 14.04) with chrome version '53.0.2763.0' & scroll bar is not getting stuck which scrolling file list in source panel, hence marking the same as TE-Verified-53.0.2763.0. Attach is the video for the same. Thank you!
,
Jun 20 2016
szager@, could you please mark this issue as 'Fixed'? Thank you!
,
Jun 21 2016
,
Jul 8 2016
Issue 624117 has been merged into this issue. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by yfulgaon...@etouch.net
, Jun 6 2016Owner: szager@chromium.org
Status: Assigned (was: Unconfirmed)
1.2 MB
1.2 MB Download
840 KB
840 KB Download
1.2 MB
1.2 MB Download