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

Issue 607432 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Regression : Unwanted white space is seen after dragging the scroll bar to RHS in devtools

Reported by mni...@etouch.net, Apr 28 2016

Issue description

Version: 52.0.2719.0 db7757b63f29696b9b465949f9704a0d9d149f2b-refs/heads/master@{#390251} (32/64-bit)
OS: Mac OS X(10.10.5, 10.11.4), Windows (7,8,8.1,10),Linux (14.04 LTS)

What steps will reproduce the problem?
1) Launch chrome, open NTP and go to devtools 
2) Now press Esc key to open console bar and click on iron icon and click on Search
3) Now resize the devtools window to RHS,scroll bar will be displayed under search tab
4) Now drag the scroll bar to RHS and observe

Actual : Unwanted white space is seen after dragging the scroll bar to RHS
Expected : Unwanted white space should not be seen after dragging the scroll bar to RHS

This is a regression issue broken in 'M-52' and below is the manual regression and Narrow bisect info:
Good build : 52.0.2718.0
Bad build : 52.0.2719.0

Narrow bisect info:
https://chromium.googlesource.com/chromium/src/+log/66bea1de4dbc4a18684655979360031c14ad97a9..f1afe5ba38f00c14f9982c8d0b3afd598865f411?pretty=fuller&n=100

Suspecting : r390177 from Narrow bisect

@luoe : Could you please help to reassign if your change is not the cause for this change.
 
Actual_video.mp4
532 KB Download
Expected_video.mp4
504 KB Download
Actual_screenshot.png
199 KB View Download
Expected_screenshot.png
195 KB View Download
Labels: ReleaseBlock-Stable
Adding RB label as this is a recent regression.

Comment 2 by l...@chromium.org, Apr 29 2016

On Linux 50.0.2661.94, I was able to reproduce the same effect, although with less whitespace than found above.  This is not a recent regression.

But a CL to fix will come!
stableoverflow.png
34.8 KB View Download

Comment 3 by l...@chromium.org, Apr 29 2016

Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, May 2 2016

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

commit 320c6f9e6c7d6104a60d7505957bfe2634a42e1f
Author: luoe <luoe@chromium.org>
Date: Mon May 02 19:52:12 2016

DevTools: fix visual regressions in full text search

BUG= 607431 , 607432 

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

[modify] https://crrev.com/320c6f9e6c7d6104a60d7505957bfe2634a42e1f/third_party/WebKit/Source/devtools/front_end/sources/sourcesSearch.css

Comment 5 by l...@chromium.org, May 2 2016

Status: Fixed (was: Started)
Changes committed.  Please review if needed.

Sign in to add a comment