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

Unified Diff: chrome/browser/resources/chromeos/chromevox/testing/chromevox_e2e_test_base.js

Issue 940763002: Reland #4: Ensure WebView notifies desktop automation on creation, (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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: chrome/browser/resources/chromeos/chromevox/testing/chromevox_e2e_test_base.js
diff --git a/chrome/browser/resources/chromeos/chromevox/testing/chromevox_e2e_test_base.js b/chrome/browser/resources/chromeos/chromevox/testing/chromevox_e2e_test_base.js
index 1edf3167bd8121ddaa27520ce6ed1162fa3bfde1..9b797cbf58a0b0536b59864ca6e1c44e0c2585a9 100644
--- a/chrome/browser/resources/chromeos/chromevox/testing/chromevox_e2e_test_base.js
+++ b/chrome/browser/resources/chromeos/chromevox/testing/chromevox_e2e_test_base.js
@@ -43,11 +43,6 @@ ChromeVoxE2ETest.prototype = {
/** @override */
testGenPreamble: function() {
GEN_BLOCK(function() {/*!
- if (chromeos::AccessibilityManager::Get()->IsSpokenFeedbackEnabled()) {
- chromeos::AccessibilityManager::Get()->EnableSpokenFeedback(false,
- ui::A11Y_NOTIFICATION_NONE);
- }
-
base::Closure load_cb =
base::Bind(&chromeos::AccessibilityManager::EnableSpokenFeedback,
base::Unretained(chromeos::AccessibilityManager::Get()),
@@ -58,11 +53,26 @@ ChromeVoxE2ETest.prototype = {
},
/**
- * Run a test with the specified HTML snippet loaded.
+ * Launch a new tab, wait until tab status complete, then run callback.
* @param {function() : void} doc Snippet wrapped inside of a function.
* @param {function()} callback Called once the document is ready.
*/
- runWithDocument: function(doc, callback) {
+ runWithLoadedTab: function(doc, callback) {
+ this.launchNewTabWithDoc(doc, function(tab) {
+ chrome.tabs.onUpdated.addListener(function(tabId, changeInfo) {
+ if (tabId == tab.id && changeInfo.status == 'complete') {
+ callback(tabId);
+ }
+ });
+ });
+ },
+
+ /**
+ * Launches the given document in a new tab.
+ * @param {function() : void} doc Snippet wrapped inside of a function.
+ * @param {function()} opt_callback Called once the document is created.
+ */
+ runWithTab: function(doc, opt_callback) {
var docString = TestUtils.extractHtmlFromCommentEncodedString(doc);
var url = 'data:text/html,<!doctype html>' +
docString +
@@ -71,13 +81,7 @@ ChromeVoxE2ETest.prototype = {
active: true,
url: url
};
- chrome.tabs.create(createParams, function(tab) {
- chrome.tabs.onUpdated.addListener(function(tabId, changeInfo) {
- if (tabId == tab.id && changeInfo.status == 'complete') {
- callback(tabId);
- }
- });
- });
+ chrome.tabs.create(createParams, opt_callback);
},
/**

Powered by Google App Engine
This is Rietveld 408576698