New issue
Advanced search Search tips

Issue 618182 link

Starred by 1 user
Status: Duplicate
Merged: issue 615308
Owner:
est...@chromium.org
Closed: Jun 2016
Cc:
ajha@chromium.org
palmer@chromium.org
benwells@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

PermissionMenuButton doesn't get a focus border.

Project Member Reported by karandeepb@chromium.org, Jun 8 2016 
Version: 53.0.2762.0
OS: All

What steps will reproduce the problem?
(1) Go to google.com
(2) Click on the green lock icon on the left of address bar to open website settings popup.
(3) Keep pressing Tab to advance focus.


What is the expected output?
PermissionMenuButton(Dropdown for permissions with label AskByDefault) should get a focus border on receiving focus.

What do you see instead?
PermissionMenuButton does not get a focus border.

This works in 51.0.2704.79 and hence is a regression.

Comment 1 by karandeepb@chromium.org, Jun 8 2016

Summary: PermissionMenuButton doesn't get a focus border. (was: PermissionMenuButton don't get a focus border.)

Comment 2 by ajha@chromium.org, Jun 9 2016

Cc: benwells@chromium.org ajha@chromium.org
Components: Blink>Focus
Labels: -Type-Bug -Pri-2 -OS-All -Needs-Bisect ReleaseBlock-Stable M-53 hasbisect OS-Linux OS-Windows Pri-1 Type-Bug-Regression
Owner: palmer@chromium.org
Status: Assigned (was: Untriaged)
Able to reproduce this on the latest canary(53.0.2762.0) on Windows-7, Linux Ubuntu 14.04. Works fine on Mac OS 10.11.5.

Regressed in M-53.

Last good build: 53.0.2749.0
First bad build: 53.0.2750.0

Changelog: https://chromium.googlesource.com/chromium/src/+log/6b4fb0f9025c4cf34374afd19a1a18da058c2796..6f7075c5e44eeb1f99f514e6e8bf53f844f41aa9

Suspected change: https://codereview.chromium.org/2006123002

palmer@: Please take a look at this.

Comment 3 by ssamanoori@chromium.org, Jun 15 2016 

Just to update:
Still able to reproduce the issue on Windows 7, Ubuntu 14.04 using latest M-53(53.0.2768.0).

palmer@Could you please look into this issue.

Comment 4 by benwells@chromium.org, Jun 15 2016

Cc: palmer@chromium.org
Owner: est...@chromium.org
This isn't palmer's change, this would be estade's (https://chromium.googlesource.com/chromium/src/+/43b34ce93a59b52dfb0f5c125a12d66b299c36d3)

estade - is this deliberate?

Comment 5 by est...@chromium.org, Jun 15 2016

Mergedinto: 615308
Status: Duplicate (was: Assigned)

Comment 6 by benhenry@chromium.org, Sep 29 2017

Components: Blink>HTML>Focus

Comment 7 by benhenry@chromium.org, Sep 29 2017

Components: -Blink>Focus
►

Sign in to add a comment