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

Unified Diff: third_party/WebKit/LayoutTests/fast/dom/shadow/tabindex-slot.html

Issue 1773553003: Removing dependency on js-test.js from shadow-dom.js (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove dependency on js-test.js from shadow-dom.js Created 4 years, 9 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/fast/dom/shadow/tabindex-slot.html
diff --git a/third_party/WebKit/LayoutTests/fast/dom/shadow/tabindex-slot.html b/third_party/WebKit/LayoutTests/fast/dom/shadow/tabindex-slot.html
index d5f73ce1691ac4f854143238b1ce5149cc213efb..6810bec91b8d65d07b770c48f28ec0d1e7dbeb5d 100644
--- a/third_party/WebKit/LayoutTests/fast/dom/shadow/tabindex-slot.html
+++ b/third_party/WebKit/LayoutTests/fast/dom/shadow/tabindex-slot.html
@@ -39,23 +39,6 @@
<script>
-function shouldCustomNavigateFocus(from, to, direction)
-{
- var fromElement = getNodeInComposedTree(from);
- if (!fromElement) {
- return;
- }
- fromElement.focus();
- if (!isInnermostActiveElement(from)) {
- return;
- }
- if (direction == 'forward')
- navigateFocusForward();
- else
- navigateFocusBackward();
- return isInnermostActiveElement(to);
-}
-
test(function() {
var xfoo = document.getElementById('x-foo');
convertTemplatesToShadowRootsWithin(xfoo);
@@ -80,10 +63,10 @@ test(function() {
];
for (var i = 0; i + 1 < elements.length; ++i)
- assert_true(shouldCustomNavigateFocus(elements[i], elements[i + 1], 'forward'), elements[i]+" to "+ elements[i+1]);
+ assert_true(shouldNavigateFocus(elements[i], elements[i + 1], 'forward'), elements[i]+" to "+ elements[i+1]);
elements.reverse();
for (var i = 0; i + 1 < elements.length; ++i)
- assert_true(shouldCustomNavigateFocus(elements[i], elements[i + 1], 'backward'), elements[i]+" to "+ elements[i+1]);
+ assert_true(shouldNavigateFocus(elements[i], elements[i + 1], 'backward'), elements[i]+" to "+ elements[i+1]);
}, 'Focus controller should treat slots as a focus scope.');
</script>

Powered by Google App Engine
This is Rietveld 408576698