Enable chrome.automation.getDesktop on all aura platforms.

chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs

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 0d336cb51994adbc2c47206ca8c89c0cdcb8d935..24f6150d8341fd99f8a9d659307df2d73feca1c7 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background_test.extjs
@@ -69,29 +69,6 @@ SYNC_TEST_F('BackgroundTest', 'NextNamespaces', function() {
   assertEquals('function', typeof(Background));
 });
 
-/** Tests that ChromeVox reads the desktop tree. */
-TEST_F('BackgroundTest', 'DesktopFocus', function() {

[dmazzoni, 2015/03/26 18:44:58]

Why are we losing this test?

[David Tseng, 2015/03/26 23:20:47]

The initial desktop tree is just the root. Until an event gets fired on
something like focus on the status tray, that's all we get. Once focus fires,
the serializer walks up to the window causing the AXAuraObjCache to cache that
window and then, find on an automation node will reach the status tray.

If you know how to focus the status tray from js, we could do that.

[dmazzoni, 2015/03/26 23:32:04]

I know we can't iterate over all windows easily, but I wonder if we could do
something in C++ code when the desktop tree is first requested, like fire an
accessibility focus event on the currently focused view?

What do you think about disabling this test for now rather than deleting it? I
really would like to see tests for things like the status tray one way or
another.

[David Tseng, 2015/03/27 17:04:38]

How about if we add a helper binding that can focus the status tray? I'm not
sure if it helps to keep a disabled test around though as long as we file a bug
for this.

I like the focus idea, but I don't think that would help in this case because we
need to have more native code mixed in among the js calls. This framework
doesn't really allow for that.

-  function findStatusTray(root) {
-    if (root.role == chrome.automation.RoleType.button &&
-        root.attributes.name &&
-        root.attributes.name.indexOf('Status tray') != -1) {
-      return root;
-    }
-    for (var i = 0; i < root.children.length; i++) {
-      var found = findStatusTray(root.children[i]);
-      if (found)
-        return found;
-    }
-    return null;
-  }
-
-  chrome.automation.getDesktop(this.newCallback(function(root) {
-    var testButton = findStatusTray(root);
-    cvox.ChromeVox.tts.expectSpeech('Status tray', this.newCallback());
-    testButton.focus();
-  }));
-});
-
 /** Tests feedback once a page loads. */
 TEST_F('BackgroundTest', 'MANUAL_InitialFeedback', function() {
   cvox.ChromeVox.tts.expectSpeech('start', this.newCallback());