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

Issue 739255 link

Starred by 1 user
Status: Fixed
Owner:
dgozman@chromium.org
Closed: Aug 2017
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : Overlapping of contents is seen in 'Add box-shadow' window after resizing devtools.

Reported by avsha...@etouch.net, Jul 5 2017 
Chrome Version : 61.0.3149.0 (Official Build) 4485eb651f6f09c3183285927fe82d17fcd7a13e-refs/heads/master@{#484159} 32/64 bit
OS : Windows(7,8,10), Linux(14.04 LTS), Mac(10.11.6, 10.12.3)

What steps will reproduce the problem?
1. Launch chrome and open devtools on NTP.
2. Drag devtools window to extreme right and in 'Styles' section, open 'Add box-shadow' window.
3. Observe.

Actual Result : Contents of 'Add box-shadow' window overlap with each other after resizing devtools.

Expected Result : Contents of 'Add box-shadow' window should not overlap and 'Inset', 'Outset' buttons should be clearly visible when devtools is resized.

This is a regression issue broken in ‘M-59’, below is the Manual Regression range and will soon update other info.
Good build : 59.0.3033.0
Bad build : 59.0.3034.0
Actual_Result.mp4
1.4 MB View Download
Expected_Result.mp4
1009 KB View Download

Comment 1 by rbasuvula@chromium.org, Jul 5 2017

Labels: hasbisect-per-revision
Owner: dgozman@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build:59.0.3033.0(Revision:455023).
Bad build:59.0.3034.0(Revision:455336).

You are probably looking for a change made after 455300 (known good), but no later than 455301 (first known bad).

CHANGE-LOG URL:
---------------
https://chromium.googlesource.com/chromium/src/+log/35cb1d3e7a0b1ebe62b485ff06bf641a01d8c36a..72b6d79734c7fd5925b6f862c588229b0207afe6

From the CL above, assigning the issue to the concern owner

@dgozman: Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.

Review-Url: https://codereview.chromium.org/2712063002
Note :Able to reproduce the issue in Win 10.0,Ubuntu 14.04 & Mac 10.12.3 and Able to reproduce in latest Canary #61.0.3149.0
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 16 2017 

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

commit 22579d77792af61fefe6d185f81f036ed3574737
Author: Dmitry Gozman <dgozman@chromium.org>
Date: Wed Aug 16 16:13:06 2017

[DevTools] Fix shadow editor layout

Bug:  739255 
Change-Id: I94b7012df8f89165c45e247498885190e0e4ac4e
Reviewed-on: https://chromium-review.googlesource.com/616220
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#494806}
[modify] https://crrev.com/22579d77792af61fefe6d185f81f036ed3574737/third_party/WebKit/Source/devtools/front_end/inline_editor/CSSShadowEditor.js
[modify] https://crrev.com/22579d77792af61fefe6d185f81f036ed3574737/third_party/WebKit/Source/devtools/front_end/inline_editor/cssShadowEditor.css

Comment 3 by dgozman@chromium.org, Aug 16 2017

Status: Fixed (was: Assigned)

Comment 4 by avsha...@etouch.net, Aug 17 2017

Labels: TE-Verified-62.0.3188.0 TE-Verified-M62
Update : 
Retested above issue in latest chrome canary #62.0.3188.0 on Windows(7,8,10), Linux(14.04 LTS), Mac(10.11.6, 10.12.3) OS and issue is fixed now and it is working as intended.
Kindly review an attached screen cast.

Thank you!
Canary_Results.mp4
1.0 MB View Download
►

Sign in to add a comment