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

Issue 633058 link

Starred by 2 users
Status: Fixed
Owner:
sammiequon@chromium.org
Closed: Aug 2016
Cc:
ajha@chromium.org
█ kavvaru@chromium.org
sammiequon@chromium.org
nyerramilli@chromium.org
█ ranjitkan@chromium.org
█ durga.behera@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: In Switch person window, focus does not move to Add person and Browse as Guest options.

Reported by vvishwak...@etouch.net, Aug 1 2016 
Version: 54.0.2815.0 (Official Build) b04cd370dcb7e737fe32d3c8a0f12daa59fa85f8-refs/heads/master@{#408889} (64-bit)
OS: Mac (10.10.5, 10.11.5)

What steps will reproduce the problem?
(1) Launch chrome and Sign in to chrome.
(2) Go to chrome://settings and add/import a supervised user from people section.
(3) Now in main user, click on avatar icon and select Switch person.
(4) Keep pressing Tab key and observe.

Focus does not move to Add person and Browse as Guest options.

Focus should move to Add person and Browse as Guest options.

This is a Regression issue broken in M-54, will soon update other info
profile_actual.mov
7.5 MB Download

Comment 1 by vvishwak...@etouch.net, Aug 1 2016

Labels: hasbisect OS-Linux OS-Windows
Owner: sammiequon@chromium.org
Status: Assigned (was: Unconfirmed)
Manual bisect
Good build: 54.0.2811.0
Bad build: 54.0.2812.0

Note: Issue is not reproducible on Chromium builds as unable to Sign in to chrome. Hence providing suspect from Changelog URL.

Changelog URL:
https://chromium.googlesource.com/chromium/src/+log/54.0.2811.0..54.0.2812.0?pretty=fuller&n=10000

Suspecting: r408734

Comment 2 by ranjitkan@chromium.org, Aug 1 2016

Cc: kavvaru@chromium.org durga.behera@chromium.org sammiequon@chromium.org ajha@chromium.org
 Issue 633138  has been merged into this issue.

Comment 3 by ranjitkan@chromium.org, Aug 1 2016

Labels: ReleaseBlock-Stable
Adding release block label, please undo if not the case.

Comment 4 by sammiequon@chromium.org, Aug 5 2016

Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 5 2016 

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

commit 442e7d64266217feb0d6299a8d31773d73775ecf
Author: sammiequon <sammiequon@chromium.org>
Date: Fri Aug 05 19:40:53 2016

Fixes keyboard event related bugs in switch person window.

BUG= 633058 , 633868 

Review-Url: https://codereview.chromium.org/2213563003
Cr-Commit-Position: refs/heads/master@{#410147}

[modify] https://crrev.com/442e7d64266217feb0d6299a8d31773d73775ecf/ui/login/account_picker/user_pod_row.js

Comment 6 by msrchandra@chromium.org, Aug 9 2016

Labels: TE-Verified-54.0.2823.0 TE-Verified-M54 TE-Verified-54.0.2824.0
Verified the issue on Latest Dev# 54.0.2824.0 on Windows & Mac and 54.0.2823.0 on Linux and is found to be fixed. Hence adding TE-Verified Labels respectively.
Also attaching a screen cast for reference.
Thank You.
633058.mov
8.1 MB Download

Comment 7 by sammiequon@chromium.org, Aug 9 2016

Status: Fixed (was: Started)
►

Sign in to add a comment