New issue
Advanced search Search tips

Issue 848637 link

Starred by 1 user
Status: WontFix
Owner:
pbos@chromium.org
Closed: Jun 2018
Cc:
nyerramilli@chromium.org
rbasuvula@chromium.org
markchang@chromium.org
mfo...@chromium.org
taku...@chromium.org
powerb@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : Grey focus highlight on 'Cast' icon in omnibox appears faint.

Reported by rp...@etouch.net, Jun 1 2018 
Chrome version: 69.0.3447.0 (Official Build) Revision ec57e30ccad35465163bffcc3860ac8b30a6fb33-refs/branch-heads/3447@{#1}(32/64-bit)
OS: Windows (7,8,8.1,10)

What steps will reproduce the problem?
1. Launch chrome,right click to open context menu and click on 'Cast' and observe 'cast' icon in omnibox
 
Actual: Grey focus highlight on 'Cast' icon appears faint
Expected: Grey focus highlight on 'Cast' icon should be seen properly

This is regression issue, broken in ‘M 69’ and will soon update other info :
Good build: 69.0.3446.0  (Revision: 563012).
Bad build: 69.0.3447.0 (Revision: 563478).
Actual_video.mp4
173 KB View Download
Expected_video.mp4
171 KB View Download
Actual_screenshot.png
115 KB View Download
Expected_screen_shot.png
122 KB View Download

Comment 1 by rp...@etouch.net, Jun 1 2018

Labels: hasbisect OS-Linux
Owner: pbos@chromium.org
Status: Assigned (was: Unconfirmed)
You are probably looking for a change made after 563438 (known good), but no later than 563401 (first known bad).

Narrow Bisect info : 
https://chromium.googlesource.com/chromium/src/+log/1ce01901eba7a11541dc1a7eac0aa6fd70ac953d..86f458cdcde115a9b14ad047b765f2be7fdb454a?pretty=fuller&n=50

Suspecting: r563398 from Narrow bisect

@pbos: Could you please help to reassign if your change is not the cause for this change.

Note:
1.Unable to provide bisect using per-revision script,Hence providing bisect with old script.
2.Issue is not seen on Mac(10.12.6,10.13.1,10.13.6)

Comment 2 by pbos@chromium.org, Jun 1 2018

Cc: markchang@chromium.org
Looks like an accidental side effect from updating Refresh (and Refresh looks fine, see screenshot). +markchang@ I think this pre-Refresh regression can be ignored for M69?
refresh-cast.png
21.6 KB View Download

Comment 3 by mfo...@chromium.org, Jun 1 2018

Status: Untriaged (was: Assigned)
Making sure this goes through triage.

Comment 4 by markchang@chromium.org, Jun 1 2018 

Yes, we should ignore this as M69 will fix this.
Should we mark as obsolete?

Comment 5 by pbos@chromium.org, Jun 4 2018

Status: WontFix (was: Untriaged)
Sure, per #4 this won't reach stable.
►

Sign in to add a comment