New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 673258 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Scrollbars in device emulation are white when overlay scrollbars enabled

Project Member Reported by jbanavatu@chromium.org, Dec 12 2016

Issue description

Chrome 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.
 
Expected.webm
1.3 MB View Download
Actual.webm
4.6 MB View Download
Labels: OS-Linux
Owner: bokan@chromium.org
Status: Assigned (was: Untriaged)
Issue is also seen in Ubuntu 14.04 on enabling 'Overlay scrollbar' flag in chrome://flags.

Bisect info:

Good build:56.0.2895.0
Bad build:56.0.2896.0

Change log Url:
https://chromium.googlesource.com/chromium/src/+log/0f2542893ccc7d1a3ec1e84dbe4895dfff04371d..3d914bde3081131a655b467c1a7a232c7c065007

@bokan : Could you please take a look at this.

Comment 2 by bokan@chromium.org, Dec 15 2016

Components: -Blink>Scroll UI>Shell
Labels: Hotlist-Input-Dev
Summary: Scrollbars in device emulation are white when overlay scrollbars enabled (was: Regression: Faint scrollbar is seen in emulated view.)
The scrollbar is still there but it's entirely white. I'd wager device emulation causes us to use the wrong scrollbar painter.
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,

Comment 4 by bokan@chromium.org, Jan 20 2017

Labels: -Pri-1 -ReleaseBlock-Stable -Type-Bug-Regression Pri-3 Type-Bug
Overlay scrollbars are turned off in 57 so this shouldn't block stable.
Cc: bokan@chromium.org
Owner: chaopeng@chromium.org
Project Member

Comment 6 by bugdroid1@chromium.org, 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

Labels: Merge-Request-59
Project Member

Comment 8 by sheriffbot@chromium.org, May 11 2017

Labels: -Merge-Request-59 Hotlist-Merge-Approved Merge-Approved-59
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

Comment 9 by gov...@chromium.org, 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.
Project Member

Comment 10 by sheriffbot@chromium.org, 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
Can you please merge this ASAP?

Comment 12 by bokan@chromium.org, 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?
Project Member

Comment 13 by sheriffbot@chromium.org, 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

Comment 14 by bokan@chromium.org, May 19 2017

Labels: -Merge-Approved-59
Status: Fixed (was: Assigned)
Removing merge-approved as the feature's been disabled in beta branch.
Status: Verified (was: Fixed)
Verified on ChromeOS 9582.0.0, 60.0.3107.0

Sign in to add a comment