Issue metadata
Sign in to add a comment
|
Regression: [MR]Focus does not traverse on 'LEARN MORE' link and 'OK GOT IT' button after pressing tab key.
Reported by
rk...@etouch.net,
Mar 28 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version: 51.0.2692.0 Revision 21ad7445a7f1b2dfa2e8ee6475594b9037b3c7c8-refs/heads/master@{#383454} OS: Mac Precondition: Enabled 'Media Router' flag from chrome://flags What steps will reproduce the problem? (1) Launch chrome,click on 'Cast' option from wrench menu or from context menu. (2) Press Tab key and observe the focus traverse. Focus does not traverse on 'LEARN MORE' link and 'OK GOT IT' button after pressing tab key i.e. it directly show on 'Cast to' drop down. Focus should show on 'LEARN MORE' link and 'OK GOT IT' button after pressing tab key. This is a regression broken in 'M-51', below is manual bisect range: Good Build:51.0.2678.0 Bad Build: 51.0.2679.0 Narrow Bisect: https://chromium.googlesource.com/chromium/src/+log/e96e24bd1b3e8e8b986d6b00fa5317ba55a0a9d2..ba7fcd6928dbcb974e28fff928e34818ad40bed6?pretty=fuller&n=100 Suspecting: r381130 Note: Issue is not seen on Windows and Linux OS.
,
Mar 28 2016
,
Apr 4 2016
,
Apr 5 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f6a6cb0dd0ffda3cdfe66c5676072cfdea25794f commit f6a6cb0dd0ffda3cdfe66c5676072cfdea25794f Author: apacible <apacible@chromium.org> Date: Tue Apr 05 17:37:39 2016 [Media Router WebUI] Move dialog load focusing to container. This ensures the first run flow is taken into account, since those elements may appear before the header elements in the DOM tree. Manually tested on Linux/Mac. BUG= 598278 Review URL: https://codereview.chromium.org/1856553002 Cr-Commit-Position: refs/heads/master@{#385213} [modify] https://crrev.com/f6a6cb0dd0ffda3cdfe66c5676072cfdea25794f/chrome/browser/resources/media_router/elements/media_router_container/media_router_container.html [modify] https://crrev.com/f6a6cb0dd0ffda3cdfe66c5676072cfdea25794f/chrome/browser/resources/media_router/elements/media_router_container/media_router_container.js [modify] https://crrev.com/f6a6cb0dd0ffda3cdfe66c5676072cfdea25794f/chrome/browser/resources/media_router/elements/media_router_header/media_router_header.html [modify] https://crrev.com/f6a6cb0dd0ffda3cdfe66c5676072cfdea25794f/chrome/browser/resources/media_router/elements/media_router_header/media_router_header.js
,
Apr 5 2016
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by ranjitkan@chromium.org
, Mar 28 2016