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

Issue 887366 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression : Tab focus is not clear around the Screenshot Notification

Project Member Reported by mmanchala@chromium.org, Sep 20

Issue description

Chrome Version: 71.0.3555.0/11081.0.0 dev channel Paine,Candy and Blaze
OS: Chrome

What steps will reproduce the problem?
1)Sign into User ->  Press 'Ctrl+F5(Overview Mode Button)' for screenshot -> Notification is seen 
(2)Now collapse it and again Expand the Notification -> Now press 'Tab' ->focus is seen on Screenshot Notification but observe focus is not around the notification 
(Please refer Video and Screenshot)

Expected: Tab focus should be seen around the Screenshot Notification
Actual: Instead tab focus is not around the notification

This is Regression Issue as same is working fine in M-69

Note : Issue is also seen on M-70
 
Actual_TabFocus.mp4
12.7 MB View Download
Actual_TabFocus.jpg
514 KB View Download
Expected_TabFocus.jpg
190 KB View Download
Owner: tetsui@chromium.org
Labels: -Pri-1 -ReleaseBlock-Stable Pri-2
Status: Assigned (was: Untriaged)
Removing RBS as cosmetic
Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 2

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

commit 580f31e2ea4b30c1b780be6975a218f5b430d7bd
Author: Tetsui Ohkubo <tetsui@chromium.org>
Date: Tue Oct 02 15:14:56 2018

Paint MessageView focus ring on top of children.

Focus ring around MessageView should be painted on top of all the
children.

TEST=manual
BUG= 887366 

Change-Id: I5667aff33196a5f3f8792b4e936e7da1cdda1057
Reviewed-on: https://chromium-review.googlesource.com/1255750
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#595843}
[modify] https://crrev.com/580f31e2ea4b30c1b780be6975a218f5b430d7bd/ui/message_center/views/message_view.cc
[modify] https://crrev.com/580f31e2ea4b30c1b780be6975a218f5b430d7bd/ui/message_center/views/message_view.h

Status: Fixed (was: Started)
Cc: yawano@google.com
 Issue 884652  has been merged into this issue.

Sign in to add a comment