Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(62)

Unified Diff: third_party/WebKit/LayoutTests/shadow-dom/focus-with-negative-index.html

Issue 1862563002: Skip shadow hosts with negative tabindex in focus navigation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Keep V0 ShadowHost behavior Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/shadow-dom/focus-with-negative-index.html
diff --git a/third_party/WebKit/LayoutTests/shadow-dom/tabindex-slot.html b/third_party/WebKit/LayoutTests/shadow-dom/focus-with-negative-index.html
similarity index 83%
copy from third_party/WebKit/LayoutTests/shadow-dom/tabindex-slot.html
copy to third_party/WebKit/LayoutTests/shadow-dom/focus-with-negative-index.html
index 335076ba823665e5a9c7443227baf883754f8836..75146fc8b1240f165de43a24464a3d071ad3734c 100644
--- a/third_party/WebKit/LayoutTests/shadow-dom/tabindex-slot.html
+++ b/third_party/WebKit/LayoutTests/shadow-dom/focus-with-negative-index.html
@@ -10,7 +10,7 @@
x-bar's shadow tree: [k1 -> k0 -> [s2]]<br>
slot #s2: [j1 -> j2 -> j3 -> j4 -> [s1] -> j0]<br><br>
slot #s1: [i1 -> i2]<br>
- <b>v1 ideal nav forward: [i0 -> j5 -> xbar -> k1 -> k0 -> j1 -> j2 -> j3 -> j4 -> i1 -> i2 -> j0 -> j6]</b><br>
+ <b>v1 ideal nav forward: [i0 -> j5 -> xbar -> k1 -> k0 -> j6]</b><br>
</p>
<input id="i0" tabindex=0 value="i0">
@@ -28,10 +28,16 @@
</div>
<template data-mode="open">
<input id="k0" tabindex=0 value="k0">
- <slot id="s2" name="s2"></slot>
+ <slot id="s2" name="s2" tabindex=-1></slot>
<input id="k1" tabindex=1 value="k1">
</template>
</div>
+ <div id="to-be-ignored-host" tabindex=-1>
+ <template data-mode="open">
+ <input id="ignored-input-in-shadow-host1" tabindex=1 value="ignored">
+ <input id="ignored-input-in-shadow-host2" tabindex=2 value="ignored">
hayato 2016/04/05 08:53:06 As we chatted offline, could you add a layout test
yuzuchan 2016/04/05 09:52:00 Done.
+ </template>
+ </div>
<input id="j6" tabindex=4 value="j6">
<input id="j5" tabindex=3 value="j5">
</template>
@@ -44,8 +50,6 @@ test(function() {
var xfoo = document.getElementById('x-foo');
convertTemplatesToShadowRootsWithin(xfoo);
var sr = xfoo.shadowRoot;
- var xbar = sr.querySelector('div');
- convertTemplatesToShadowRootsWithin(xbar);
var elements = [
'i0',
@@ -53,13 +57,6 @@ test(function() {
'x-foo/x-bar',
'x-foo/x-bar/k1',
'x-foo/x-bar/k0',
- 'x-foo/j1',
- 'x-foo/j2',
- 'x-foo/j3',
- 'x-foo/j4',
- 'i1',
- 'i2',
- 'x-foo/j0',
'x-foo/j6'
];

Powered by Google App Engine
This is Rietveld 408576698