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

Issue 623524 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression

Blocked on:
issue 626134



Sign in to add a comment

Regression: Unnecessary scroll bar is seen in 'Application' section after clicking on 'Undock into separate window' option.

Reported by rk...@etouch.net, Jun 27 2016

Issue description

Chrome Version: 53.0.2780.0 Revision 5453dc6abc2d7c797c7f9f16df8aed8e6358a217-refs/heads/master@{#402053}(32/64 bit)
OS: Windows(7,8,10), Mac(10.10.5, 10.11.4), Linux(14.04 LTS)

What steps will reproduce the problem?
(1) Launch chrome, open dev tool window and click on 'Customize and control Devtools' then click on 'Dock to bottom' option.
(2) Again click on 'Customize and control Devtools' then click on 'Undock into separate window' option.
(3) Navigate to 'Application' section and observe.

Unnecessary scroll bar is seen in 'Application' section after clicking on 'Undock into separate window' option.

Scroll bar should not be seen in 'Application' section after clicking on 'Undock into separate window' option.

This is a regression issue, broken in 'M-53', below is bisect info:

Good Build: 53.0.2778.0
Bad Build: 53.0.2780.0

Narrow Bisect:
https://chromium.googlesource.com/chromium/src/+log/90c3a38a3644e22e98924fca3f4f9172d82cb366..4cc4f198b160f22dc7484e16d7b3d341dbe7fca2?pretty=fuller&n=100

Suspecting: r401745

 
Scroll_Actual.mp4
813 KB View Download
Scroll_Expected.mp4
795 KB View Download
Labels: ReleaseBlock-Stable
Adding RB label as this is a recent regression
Just to update the bug, Issue is still repro on latest chrome canary M53-53.0.2783.2. Observed Unnecessary scroll bar under 'Application' section.

szager@ - Hey, Will you mind looking int this issue since it is marked as RB-Stable, If it's not related to your change please feel free to reassign it to the concerned Dev person.

Thanks!
@szager: Gentle Ping! Would you mind providing an update on the above issue as per comment#2 ?

I really appreciate your response.

Thank you!
Owner: l...@chromium.org

Comment 5 by l...@chromium.org, Jul 6 2016

Blockedon: 626134

Comment 6 by l...@chromium.org, Jul 6 2016

I've filed a separate  issue #626134  with what I believe is the root cause here, and added @szager.
Project Member

Comment 7 by bugdroid1@chromium.org, Jul 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/11e29c02fa751ee9411260e5ca662622cbd679ca

commit 11e29c02fa751ee9411260e5ca662622cbd679ca
Author: szager <szager@chromium.org>
Date: Thu Jul 07 00:27:23 2016

Get rid of spurious horizontal scrollbars on column flexboxes.

BUG= 623524 
R=cbiesinger@chromium.org,eae@chromium.org

Review-Url: https://codereview.chromium.org/2116873002
Cr-Commit-Position: refs/heads/master@{#404008}

[modify] https://crrev.com/11e29c02fa751ee9411260e5ca662622cbd679ca/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp

Comment 8 by l...@chromium.org, Jul 7 2016

Cc: szager@chromium.org
Status: Fixed (was: Assigned)
Thanks for the super quick fix, szager@
Project Member

Comment 9 by bugdroid1@chromium.org, Jul 11 2016

Labels: merge-merged-2785
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e0501af4358aade5483ac93a668ab104fba75eb6

commit e0501af4358aade5483ac93a668ab104fba75eb6
Author: szager <szager@chromium.org>
Date: Mon Jul 11 19:48:25 2016

Get rid of spurious horizontal scrollbars on column flexboxes.

Merged from: https://codereview.chromium.org/2116873002

BUG= 623524 
TBR=cbiesinger@chromium.org,eae@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2138193002
Cr-Commit-Position: refs/branch-heads/2785@{#91}
Cr-Branched-From: 68623971be0cfc492a2cb0427d7f478e7b214c24-refs/heads/master@{#403382}

[modify] https://crrev.com/e0501af4358aade5483ac93a668ab104fba75eb6/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.cpp

Comment 10 by ajha@chromium.org, Jul 12 2016

Labels: TE-Verified-M53 TE-Verified-53.0.2785.14
Verified the merge on the latest M-53(53.0.2785.14, 2785@{#96}). This is working as intended on Windows-7, Mac OS 10.11.5 and Linux Ubuntu 14.04. Attached is the screen-shot.
623524.png
116 KB View Download

Sign in to add a comment