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

Issue 793225 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Regression: [DevTools] Second bidirectional arrow is misplaced on ColorPicker window.

Reported by dchau...@etouch.net, Dec 8 2017

Issue description

Chrome Version: 65.0.3288.0 (Official Build)25b7dd1729eebe969f4bfd358babec31b96589ca-refs/heads/master@{#522666} 32/64-bit.
OS: Win(7,8,10), Mac OS X(10.12.6,10.13.2) and Linux (14.04 LTS).

What steps will reproduce the problem?
1. Launch Chrome and open DevTools window.
2. Click to open ColorPicker window under Styles section.
3. Now observe the second bidirectional arrow.

Second bidirectional arrow is misplaced.
Second bidirectional arrow should not misplace.

This is a regression issue, broken in M-65 series, below is manual regression range.

Good build: 65.0.3284.0
Bad build: 65.0.3285.0

You are probably looking for a change made after 521548 (known good), but no later than 521549 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspect as some perf builds might get missing due to failure.
  https://chromium.googlesource.com/chromium/src/+log/65b472737b0c1100ee672b9b89c3d2439defb654..09f677bd1562c40825b936f2af6c4d2232a3d585

Suspect: https://chromium.googlesource.com/chromium/src/+/09f677bd1562c40825b936f2af6c4d2232a3d585

@aboxhall: Kindly help to reassign, if your changes are not related to this issue.

Kindly review the attached screenshot for reference.
 
ColorPicker_Screenshot.png
83.9 KB View Download
Actual behavior.mp4
640 KB View Download
Expected behavior.mp4
291 KB View Download
Labels: ReleaseBlock-Stable
Adding RB Label as this is a recent Regression. Please remove if not required.
Thank You.
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 12 2017

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

commit 8dd06837e17b1a41d4f129976ca6b400b4721a97
Author: Alice Boxhall <aboxhall@chromium.org>
Date: Tue Dec 12 21:09:03 2017

[Devtools] Move palette switcher button back up to the top

Bug:  793225 
Change-Id: I888f166d099b7829b942fc8d31c16fec8a3df411
Reviewed-on: https://chromium-review.googlesource.com/821950
Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Commit-Queue: Alice Boxhall <aboxhall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523548}
[modify] https://crrev.com/8dd06837e17b1a41d4f129976ca6b400b4721a97/third_party/WebKit/Source/devtools/front_end/color_picker/spectrum.css

Status: Started (was: Assigned)

Comment 4 by dchau...@etouch.net, Dec 13 2017

Update:-
 Tested this issue on Windows (7,8,10) and Mac OS X(10.12.6,10.13.2) machines using latest Chrome canary build# 65.0.3293.0 fix is working as expected. 

NOTE: Linux build is not yet available, so not adding TE-Verified labels

Please find the attached screen-cast for reference.

Thanks..!
LatestCanary behavior.mp4
378 KB View Download
Status: Fixed (was: Started)

Comment 6 by dchau...@etouch.net, Dec 14 2017

Labels: TE-Verified-M65 TE-Verified-65.0.3294.0
Update:-
 Tested this issue on Windows (7,8,10), Linux(14.04 LTS) and Mac OS X(10.12.6,10.13.2) machines using latest Chrome canary build# 65.0.3294.0 fix is working as expected.

Thanks

Sign in to add a comment