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

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

Issue 2388193003: Always enqueue live regions from background tabs (Closed)
Patch Set: Remove obsolete helper. Created 4 years, 2 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/cvox2/background/automation_util_test.extjs
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_util_test.extjs b/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_util_test.extjs
index 7461eca403c375385e2ab653c65b61e00c18ad40..40427225dbf7b93b36063f8d9d634974f3c57a2f 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_util_test.extjs
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/automation_util_test.extjs
@@ -137,62 +137,6 @@ TEST_F('AutomationUtilE2ETest', 'GetDirection', function() {
});
});
-TEST_F('AutomationUtilE2ETest', 'IsInSameWebpage', function() {
- this.runWithLoadedTree(this.basicDoc, function(root) {
- this.runWithLoadedTree(this.secondDoc, function(root2) {
- chrome.automation.getDesktop(this.newCallback(function(desktop) {
- assertTrue(AutomationUtil.isInSameWebpage(root, root));
- assertTrue(AutomationUtil.isInSameWebpage(root.firstChild, root));
- assertTrue(AutomationUtil.isInSameWebpage(root, root.firstChild));
-
- assertTrue(AutomationUtil.isInSameWebpage(root2, root2));
- assertTrue(AutomationUtil.isInSameWebpage(root2.firstChild, root2));
- assertTrue(AutomationUtil.isInSameWebpage(root2, root2.firstChild));
-
- assertFalse(AutomationUtil.isInSameWebpage(root, root2));
- assertFalse(AutomationUtil.isInSameWebpage(root.firstChild, root2));
- assertFalse(AutomationUtil.isInSameWebpage(root2.firstChild));
- assertFalse(AutomationUtil.isInSameWebpage(
- root.firstChild, root2.firstChild));
-
- assertFalse(AutomationUtil.isInSameWebpage(root, desktop));
- assertFalse(AutomationUtil.isInSameWebpage(root2, desktop));
- }.bind(this)));
- }.bind(this));
- }.bind(this));
-});
-
-TEST_F('AutomationUtilE2ETest', 'IsInSameWebpageIframe', function() {
- // Wait for load complete on both outer frame and iframe.
- var outerFrame;
- var innerFrame;
- var desktop;
- var onSuccess = this.newCallback(function() {
- assertTrue(AutomationUtil.isInSameWebpage(outerFrame, innerFrame));
- assertFalse(AutomationUtil.isInSameWebpage(outerFrame, desktop));
- assertFalse(AutomationUtil.isInSameWebpage(innerFrame, desktop));
- assertFalse(AutomationUtil.isInSameWebpage(outerFrame.parent, innerFrame));
- }.bind(this));
-
- chrome.automation.getDesktop(function(r) {
- desktop = r;
- this.runWithTab(this.iframeDoc, function(newTabUrl) {
- var listener = function(evt) {
- if (evt.target.docUrl == newTabUrl)
- outerFrame = evt.target;
- if (evt.target.docUrl.indexOf('data:text/html') == 0)
- innerFrame = evt.target;
-
- if (outerFrame && innerFrame) {
- desktop.removeEventListener('loadComplete', listener, true);
- onSuccess();
- }
- }.bind(this);
- desktop.addEventListener('loadComplete', listener, true);
- }.bind(this));
- }.bind(this));
-});
-
TEST_F('AutomationUtilE2ETest', 'VisitContainer', function() {
this.runWithLoadedTree(toolbarDoc, function(r) {
var pred = function(n) { return n.role != 'rootWebArea'; };

Powered by Google App Engine
This is Rietveld 408576698