Scrollbars in device emulation are white when overlay scrollbars enabled |
||||||||
Issue descriptionChrome Version: 57.02948.0/9075.0.0 dev channel Falco,Spring, Kip OS: Chrome OS What steps will reproduce the problem? (1)Launch chrome>> Navigate to NTP or any page>Open devtools using Ctrl+Shift+I and emulate (2)Now Double tap on the page such that it gets zoomed in emulation view and Observe scrollbar. Expected: Scrollbars should be clearly visible in emulation view. Actual: Instead faint scrollbar is seen which gives an impression as if there is scrollbar missing. This is regression issue as it is working fine in 55.0.2883.87/8872.70.0 stable channel peppy. Attaching screen-cast for reference.
,
Dec 15 2016
The scrollbar is still there but it's entirely white. I'd wager device emulation causes us to use the wrong scrollbar painter.
,
Jan 5 2017
Just to update, still able to reproduce the issue on Linux Ubuntu using chrome version 57.0.2970.0 bokan@ could you please look into this issue. Thanks,
,
Jan 20 2017
Overlay scrollbars are turned off in 57 so this shouldn't block stable.
,
May 9 2017
,
May 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b50f6446eb79c4aaaac0a5eefdf90d2b4a99ce54 commit b50f6446eb79c4aaaac0a5eefdf90d2b4a99ce54 Author: chaopeng <chaopeng@chromium.org> Date: Wed May 10 14:41:05 2017 DevTools shows overlay scrollbar for enabled aura overlay scrollbar and mac DevTools mobile emulator currently does not show android overlay scrollbar when we enabled aura overlay scrollbar or we use it on Mac. This issue is caused by we have not set any overlay scrollbar setting on Mac and have not set solid_color_scrollbar_color when enabled aura overlay scrollbar. In this patch, we set default overlay scrollbar same as Android Overlay Scrollbar for Android and DevTools mobile emulator, Aura Overlay Scrollbar will override the default settings. BUG= 673258 Review-Url: https://codereview.chromium.org/2870733002 Cr-Commit-Position: refs/heads/master@{#470570} [modify] https://crrev.com/b50f6446eb79c4aaaac0a5eefdf90d2b4a99ce54/content/renderer/gpu/render_widget_compositor.cc
,
May 11 2017
,
May 11 2017
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 12 2017
Please merge your change to M59 branch 3071 by 4:00 PM PT, Monday (05/15) so we can take it in for next week beta release. Thank you.
,
May 15 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 17 2017
Can you please merge this ASAP?
,
May 17 2017
The merges have been conflicting and so we've decided to push the launch of the feature back to M60. So this no longer needs a merge, do we need to change the labels?
,
May 19 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 19 2017
Removing merge-approved as the feature's been disabled in beta branch.
,
May 24 2017
Verified on ChromeOS 9582.0.0, 60.0.3107.0 |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by jbanavatu@chromium.org
, Dec 12 2016Owner: bokan@chromium.org
Status: Assigned (was: Untriaged)