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

Issue 653401 link

Starred by 1 user
Status: Fixed
Owner:
dpa...@chromium.org
Closed: Oct 2016
Cc:
gov...@chromium.org
nyerramilli@chromium.org
pbomm...@chromium.org
█ tkonch...@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression: Default highlight is not seen on drop down option of chrome://md-settings/siteSettings/all.

Reported by rk...@etouch.net, Oct 6 2016 
Chrome Version: 55.0.2882.0 Revision 03e86d027151639ad79c2074199f4e75beab0674-refs/heads/master@{#423378} 
OS:Windows (7,8,8.1,10)

What steps will reproduce the problem?
(1) Launch chrome, navigate to https://permission.site and click on 'Location' and 'Allow' it.
(2) Then navigate to chrome://md-settings/siteSettings/all and click on iron icon of 'permission.site'
(3) observe .

Actual: Default highlight is not seen on drop down option.

Expected: Default highlight should seen seen on drop down option.

This is a regression issue, broken in 'M-55', will soon update the other info:

Good Build: 55.0.2881.0
Bad Build: 55.0.2882.0
Actual_Highlight.mp4
361 KB View Download
Expected_Highlight.mp4
327 KB View Download

Comment 1 by tkonch...@chromium.org, Oct 7 2016

Cc: dpa...@chromium.org
Labels: Proj-MaterialDesign-WebUI hasbisect-per-revision
Owner: finnur@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,

Good build:55.0.2881.0 (revision : 423030)
Bad build:55.0.2882.0 (revision : 423378)


You are probably looking for a change made after 423227 (known good), but no later than 423228 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspectas some perf builds might get missing due to failure.
  https://chromium.googlesource.com/chromium/src/+log/c4e50671c1d6124c8cc4b315f9308a365da79df4..0f6bea6b3b13dc044d18813348c1137d0d0b51bc

finnur@, Could you please reassign if this is not related to your change.

Comment 2 by dschuyler@chromium.org, Oct 11 2016

Owner: dschuyler@chromium.org
Moving site settings bugs from finnur@ to dschuyler@.

Comment 3 by dschuyler@chromium.org, Oct 11 2016

Owner: dpa...@chromium.org
dpapad@, is this an action menu thing (rather than a site settings thing)?
Does the action menu changes address this?

Comment 4 by dpa...@chromium.org, Oct 11 2016 

Yes, this will be addressed. The first focusable option in the menu will be focused.

Comment 5 by dpa...@chromium.org, Oct 13 2016

Labels: -Pri-1 Pri-2

Comment 6 by dpa...@chromium.org, Oct 21 2016

Status: Fixed (was: Assigned)
Fixed by https://codereview.chromium.org/2438643004.

Comment 7 by ligim...@chromium.org, Oct 25 2016

Cc: pbomm...@chromium.org gov...@chromium.org
rkote@ please verify in the latest canary.
We may need a merge to M55 also.

Comment 8 by dpa...@chromium.org, Oct 25 2016 

MD Settings has not launched. Why would there be a need for merging?
►

Sign in to add a comment