Implement automatic load of composed/embedded automation trees

chrome/common/extensions/api/automation.idl

Index: chrome/common/extensions/api/automation.idl
diff --git a/chrome/common/extensions/api/automation.idl b/chrome/common/extensions/api/automation.idl
index 2c48253a86f695ee4cf2d8bf473f5cb6924041dc..bee0460133959c543ce314dc81c55b6455e4b9d7 100644
--- a/chrome/common/extensions/api/automation.idl
+++ b/chrome/common/extensions/api/automation.idl
@@ -285,6 +285,14 @@
     // Sets selection within a text field.
     static void setSelection(long startIndex, long endIndex);
 
+    // Whether this AutomationRootNode is loaded or not. If false, call load()
+    // to get the contents. Only applicable on webArea nodes.
+    boolean loaded;

[dmazzoni, 2014/10/29 06:53:30]

Should this be boolean? so that it can be undefined on other nodes?

+
+    // Load the accessibility tree for this AutomationRootNode. Only
+    // applicable on webArea nodes.
+    static void load(RootCallback callback);

[dmazzoni, 2014/10/29 06:53:30]

can a method be optional?

+
     // Adds a listener for the given event type and event phase.
     static void addEventListener(
         EventType eventType, AutomationListener listener, boolean capture);
@@ -313,15 +321,6 @@
   //
   // Extends $(ref:automation.AutomationNode).
   [nocompile, noinline_doc] dictionary AutomationRootNode {
-    // TODO(aboxhall/dtseng): implement loading. Kept separate to not include
-    // in generated docs.
-
-    // Whether this AutomationRootNode is loaded or not. If false, call load()
-    // to get the contents.
-    boolean loaded;
-
-    // Load the accessibility tree for this AutomationRootNode.
-    static void load(RootCallback callback);
   };
 
   interface Functions {