Regression: On NTP, focus issue is seen on pressing Shift+Tab key.
Reported by
vvishwak...@etouch.net,
Sep 26 2016
|
||||||
Issue descriptionVersion: 55.0.2872.0 (Official Build) 5264ac775e58852ece697a6e26117d72181c4f41-refs/heads/master@{#420859} (32/64-bit) OS: Windows (7,8,10), Mac(10.10.5, 10.11.4), Linux (14.04 LTS) What steps will reproduce the problem? 1) Launch chrome and Sign in to chrome. 2) Open a NTP and click on Google notification button on right side. 3) Press Tab key thrice (focus is on ‘Previously read’ option) and then press Tab key once. 4) Now press Shift+Tab key and observe. On pressing Shift+Tab key focus moves to Settings button. On pressing Shift+Tab key focus should be on ‘Previously read’ option. This is a Regression issue broken in M-50, below is the bisect info Manual bisect: Good build: 50.0.2657.0 Bad build: 50.0.2658.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/50.0.2657.0..50.0.2658.0?pretty=fuller&n=10000 Suspecting: r376949
,
Sep 27 2016
Smaller reproduction:
<!DOCTYPE html>
<style>
iframe:focus {
outline: blue auto 5px;
}
</style>
<p>TAB, Shift-TAB should focus on "previously."</p>
<input id=outer1 onfocus="rejectFocus()">
<iframe srcdoc="<input id=inner1 value=settings><input id=inner2 autofocus value=previously>"></iframe>
<input id=outer2 onfocus="rejectFocus()">
<script>
function rejectFocus() {
document.querySelector('iframe').focus();
}
</script>
,
Sep 27 2016
,
Sep 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bd983452a314772dd8409979f3deda637336f0ca commit bd983452a314772dd8409979f3deda637336f0ca Author: tkent <tkent@chromium.org> Date: Tue Sep 27 06:03:57 2016 Sequential focus navigation across frames should clear per-document starting point. BUG= 650228 Review-Url: https://codereview.chromium.org/2377543002 Cr-Commit-Position: refs/heads/master@{#421121} [modify] https://crrev.com/bd983452a314772dd8409979f3deda637336f0ca/third_party/WebKit/LayoutTests/fast/events/sequential-focus-navigation-starting-point.html [modify] https://crrev.com/bd983452a314772dd8409979f3deda637336f0ca/third_party/WebKit/Source/core/page/FocusController.cpp
,
Sep 27 2016
,
Sep 29 2017
,
Sep 29 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by shrike@chromium.org
, Sep 26 2016