New issue
Advanced search Search tips

Issue 821143 link

Starred by 4 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Mac
Pri: 2
Type: Bug



Sign in to add a comment

[root layer scrolls] Incorrect width of the extension popup with vertical overflow

Reported by woxxom@gmail.com, Mar 12 2018

Issue description

Observed in Windows 7 x64 with:
  Chrome 66 dev
  Chrome 67 canary

1. install the attached extension
2. click its toolbar icon

Expected: a) yellow background fills the entire width and b) scrollbar is shown
Observed: in addition to a) and b) there is a red column on the right

===================================================

Bisect info: 535635 (good) - 535645 (bad)
https://chromium.googlesource.com/chromium/src/+log/e04a14f4..004f3b4b?pretty=fuller
Suspecting r535636 = 96f85b68747a679ea1ac4cd05d6743ae5f7142b7 = https://crrev.com/c/857902 by skobes@chromium.org
"Enable root layer scrolling."
Landed in 66.0.3345.0

===================================================

Confirmed by disabling the suspected CL and observing the bug is gone:
chrome --disable-blink-features=RootLayerScrolling

===================================================

The popup is just a tall div, nothing fancy:
<!DOCTYPE html>
<html>
  <body style="width:200px; margin:0; background-color:red">
    <div style="height:50vh; background-color:lightyellow">
      In bad builds you'll see a red column on the right.
    </div>
    <div style="height:200vh; background-color:yellow"></div>
  </body>
</html>
 
test-ext.zip
626 bytes Download
GOOD.png
4.9 KB View Download
BAD.png
4.8 KB View Download

Comment 1 by woxxom@gmail.com, Mar 12 2018

It just occurred to me to run a bisect with the RLS feature forced:
chrome --enable-blink-features=RootLayerScrolling --root-layer-scrolls 

Bisect info: 397111 (good) - 397120 (bad)
https://chromium.googlesource.com/chromium/src/+log/1a3db4d1..b34eeae4?pretty=fuller
Suspecting r397112 = c9b3e99e2a71f7bcc4d9652916347782ff616e0d = https://crrev.com/1980103002 by szager@chromium.org
"Include auto vertical scrollbar in intrinsicScrollbarLogicalWidth."
Landed in 53.0.2756.0

The suspected r397112 didn't cause the bug to manifest before Chrome 66 where RootLayerScrolling was enabled by default.
Labels: Needs-Triage-M66
Labels: -Pri-3 Triaged-ET M-67 Target-67 FoundIn-67 OS-Linux Pri-2
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on Win-10 and Ubuntu 14.04 using chrome version #64.0.3282.186 and latest canary #67.0.3368.1. Issue is not seen in OS-mac.
This is a non-regression issue as it is observed from M60 old builds. 

Hence, marking it as untriaged to get more inputs from dev team.

Thanks...!!
Cc: dpa...@chromium.org
Components: Blink
Owner: szager@chromium.org
Status: Assigned (was: Untriaged)
From the bisect in #1 (thanks woxxom!), assigning to szager@.
Components: -Blink Blink>Layout
layout I guess, per the bisect above

Comment 6 by woxxom@gmail.com, Mar 22 2018

Here's a workaround which might be useful for those who stumbles upon this bug:

html {
  overflow: overlay;
}

Comment 7 by pdr@chromium.org, May 9 2018

Labels: OS-Mac
This still reproduces. Ping for fix?
Cc: vamshi.kommuri@chromium.org
 Issue 886579  has been merged into this issue.

Sign in to add a comment