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

Issue 626134 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 623524



Sign in to add a comment

Regression: Unnecessary scrollbar appears

Project Member Reported by l...@chromium.org, Jul 6 2016

Issue description

Version: Chrome 54.0.2790.0
OS: Linux64

What steps will reproduce the problem?
(1) Open the attached scrollinvalidate.html file

What is the expected output?
No horizontal scrollbar should appear, just a vertical one.  It works fine on stable.

What do you see instead?
Observe that a horizontal scrollbar appears in the div#box with overflow: auto.  It appears that the scrollbar's 'thumb' takes up the full width.

If you click on the 'togglePadding' button twice, the div updates and the horizontal scrollbar no longer appears.


Suspecting commit & associated bug: 
https://chromium.googlesource.com/chromium/src/+/96c111dfbb085888c163c987375ff854b4619c5e
https://bugs.chromium.org/p/chromium/issues/detail?id=621644
 
scrollinvalidate.html
747 bytes View Download
Labels: ReleaseBlock-Stable M-53
Status: Assigned (was: Untriaged)
Labels: Merge-Request-53
Status: Fixed (was: Assigned)
Fixed by https://codereview.chromium.org/2116873002/
Cc: nyerramilli@chromium.org szager@chromium.org tkonch...@chromium.org rnimmagadda@chromium.org
 Issue 624749  has been merged into this issue.
Cc: durga.behera@chromium.org brajkumar@chromium.org ajha@chromium.org kavvaru@chromium.org
 Issue 625622  has been merged into this issue.
Cc: ssamanoori@chromium.org
 Issue 625056  has been merged into this issue.

Comment 6 by dimu@google.com, Jul 9 2016

Labels: -Merge-Request-53 Merge-Approved-53 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M53 (branch: 2785)
Project Member

Comment 7 by sheriffbot@chromium.org, Jul 13 2016

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 8 by szager@chromium.org, Jul 13 2016

Labels: -Merge-Approved-53

Sign in to add a comment