Regression: Scrollbar is seen missing in devtools when page zoom >110 % |
|||||||
Issue descriptionChrome Version: 58.0.3004.0 dev OS: Ubuntu 14.04,Windows What steps will reproduce the problem? (1)Go to chrome://settings and change page zoom to 125%/150% (2)Now open devtools on any page >> Hit ctrl+p so that source list appears >> Observe for scrollbar Expected: Scrollbar should be seen and scrolling with mouse wheel should be smooth. Actual: Instead scrollbar is seen missing and mouse wheel scroll is not smooth. This is a regression issue broken in M57. Good Build: 57.0.2979.0 dev Bad Build: 57.0.2980.0 dev
,
Feb 6 2017
as this is recent regression marking as RB stable, please change accordingly.
,
Feb 6 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/35ab0f010dd7ca82e59c9d31114e033780e4b9e6 commit 35ab0f010dd7ca82e59c9d31114e033780e4b9e6 Author: einbinder <einbinder@chromium.org> Date: Mon Feb 06 23:54:35 2017 DevTools: Show scrollbars on a small FilteredListWidget BUG= 688933 Review-Url: https://codereview.chromium.org/2680543003 Cr-Commit-Position: refs/heads/master@{#448446} [modify] https://crrev.com/35ab0f010dd7ca82e59c9d31114e033780e4b9e6/third_party/WebKit/Source/devtools/front_end/quick_open/FilteredListWidget.js [modify] https://crrev.com/35ab0f010dd7ca82e59c9d31114e033780e4b9e6/third_party/WebKit/Source/devtools/front_end/quick_open/filteredListWidget.css [modify] https://crrev.com/35ab0f010dd7ca82e59c9d31114e033780e4b9e6/third_party/WebKit/Source/devtools/front_end/ui/Dialog.js [modify] https://crrev.com/35ab0f010dd7ca82e59c9d31114e033780e4b9e6/third_party/WebKit/Source/devtools/front_end/ui/GlassPane.js
,
Feb 7 2017
einbinder@ If possible, please merge your change to M57 branch 2987.
,
Feb 8 2017
A friendly reminder that M57 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
,
Feb 14 2017
Rechecked the issue on canary version 58.0.3012.0 on Windows 10, Ubuntu 14.04, MAC 10.12.2, fix is working as intended. Able to scroll the page and scroll bar is displayed. Adding TE-Verified labels.
,
Feb 16 2017
A friendly reminder that M57 Stable is launch is coming VERY soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch (2987) ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!
,
Feb 22 2017
URGENT - PTAL ASAP. We're getting VERY close to M57 Stable promotion. And this issue is marked as M57 stable release blocker. Pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Know that this issue shouldn't block the release? Remove the ReleaseBlock-Stable label or move to M58. Thank you.
,
Feb 23 2017
einbinder@, please request a merge to M57 if CL listed at #3 is a safe merge to M57. Thank you.
,
Feb 27 2017
Doesn't seem important enough to merge.
,
Feb 27 2017
[Auto-generated comment by a script] We noticed that this issue is targeted for M-57; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-57 label, otherwise remove Merge-TBD label. Thanks.
,
Feb 27 2017
Per comment #10, M57 merge is not needed. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by rbasuvula@chromium.org
, Feb 6 2017Labels: -Needs-Bisect hasbisect-per-revision OS-Mac
Owner: einbinder@chromium.org
Status: Assigned (was: Unconfirmed)