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

Unified Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs

Issue 2479893002: Revert of Use setSequentialFocusNavigationStartingPoint in ChromeVox (Closed)
Patch Set: Created 4 years, 1 month 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: chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
index 47becb01d34d3b25181f281fea1f75797129fa27..9e13d94a0f782b74c11ef52a903930cb8b62ef3b 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
@@ -573,7 +573,7 @@
}.bind(this));
});
-TEST_F('BackgroundTest', 'ShouldNotFocusIframe', function() {
+TEST_F('BackgroundTest', 'FocusIframe', function() {
this.runWithLoadedTree( function() {/*!
<iframe tabindex=0 src="data:text/html,<p>Inside</p>"></iframe>
<button>outside</button>
@@ -585,35 +585,14 @@
assertEquals('button', button.role);
var didFocus = false;
- iframe.addEventListener('focus', function() {
+ iframe.focus = function() {
didFocus = true;
- });
+ };
var b = ChromeVoxState.instance;
b.currentRange_ = cursors.Range.fromNode(button);
doCmd('previousElement');
assertFalse(didFocus);
}.bind(this));
-});
-
-TEST_F('BackgroundTest', 'ShouldFocusLink', function() {
- this.runWithLoadedTree( function() {/*!
- <div><a href="#">mylink</a></div>
- <button>after</button>
- */}, function(root) {
- var link = root.find({role: RoleType.link});
- var button = root.find({role: RoleType.button});
-
- assertEquals('link', link.role);
- assertEquals('button', button.role);
-
- var didFocus = false;
- link.addEventListener('focus', this.newCallback(function() {
- // Success
- }));
- var b = ChromeVoxState.instance;
- b.currentRange_ = cursors.Range.fromNode(button);
- doCmd('previousElement');
- });
});
TEST_F('BackgroundTest', 'NoisySlider', function() {

Powered by Google App Engine
This is Rietveld 408576698