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

Issue 609397 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Regression : Grey highlight on Bookmark bar folder appears chopped after resizing the separator line in chrome://bookmarks

Reported by mni...@etouch.net, May 5 2016

Issue description

Version: 52.0.2725.0 3bfc96772d01d42a50683cae507a53cf07bb606d-refs/heads/master@{#391707} (32/64-bit)
OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.10.5, 10.11.4)
URL : chrome://bookmarks/#1

What steps will reproduce the problem?
1) Launch chrome and navigate to above URL.
2) Now click on 'Organize' drop down list and add page then click on 'Folders' and add a new folder
3) Now click on 'Bookmark bar' folder and open devtools and emulated view and select 'Responsive' mode
4) Now drag the devtools window to LHS and resize the separator line in bookmark page to LHS,observe the Bookmark bar folder highlight

Actual : Grey highlight on Bookmark bar folder appears chopped after resizing the separator line in chrome://bookmarks page
Expected : Grey highlight on Bookmark bar folder should be seen properly after resizing the separator line in chrome://bookmarks page

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/4652931e184be34512927790d6e53ae8c32bbb43..aae4342bd58e7aaba86bfc80aa4d7157c75ffd53?pretty=fuller&n=50

Suspecting : r390215 from Narrow bisect

@rego : Could you please help to reassign if your change is not the cause for this change.
 
Actual_video.mp4
1.9 MB Download
Expected_video.mp4
1.4 MB Download
Actual_screenshot.png
217 KB View Download
Expected_screenshot.png
224 KB View Download
Labels: ReleaseBlock-Stable
Marking the above issue as RB-Stable, as this is recent regression.

Feel free to remove if someone feels otherwise.

Thank you!

Comment 2 by r...@igalia.com, May 5 2016

Did you have the "Experimental Web Platform Features" flag enabled,
or are you running chrome using --enable-experimental-web-platform-features?

If that's not the case the Grid Layout code in my patch shouldn't be executed.

In addition I don't see that the page is using Grid Layout at all,
so even with the flag enabled my patch shouldn't affect this.

Comment 3 by mni...@etouch.net, May 5 2016

With response to comment #2 :
Rechecked the above issue after enabling the flag 'Experimental Web Platform Features' and the issue is still reproduced.

Note : This issue is reproducible in both cases i.e on enabling or disabling the flag.

Comment 4 by r...@igalia.com, May 5 2016

So probably it's unrelated to my patch, as it only modifies code that is run when the runtime flag is on.
Labels: -ReleaseBlock-Stable
just to update, unable to reproduce the issue on Win7, Mac OS X 10.11.4, Ubuntu 14.04 using Canary #52.0.2734.0

removing blocker label, feel free to add if required.

Comment 6 by r...@igalia.com, May 13 2016

Cc: r...@igalia.com
Owner: ----
Status: Unconfirmed (was: Assigned)
As I said, it seems completely unrelated to my grid change, so I'm removing myself from owner field.

Also from @nyerramilli comment it seems to be fixed now,
@mnikam are you able to reproduce it yet with the last canary?

Comment 7 by caseq@chromium.org, May 13 2016

Components: -Platform>DevTools
Labels: Needs-Feedback

Comment 9 by mni...@etouch.net, May 16 2016

Labels: -Needs-Feedback
With response to comment #6 :
Rechecked the above issue on latest canary chrome version : 52.0.2737.0 with OS Win7, Mac OS X 10.11.4, Ubuntu 14.04 and the issue is not reproducible.Please refer the attached screenshot for your reference.
Actual_video_canaray_52.0.2737.mp4
2.1 MB Download

Comment 10 by r...@igalia.com, May 16 2016

Status: WontFix (was: Unconfirmed)
@mnikam thanks for checking it, I guess we can close it then.

Sign in to add a comment