Implement automatic load of composed/embedded automation trees

chrome/browser/extensions/api/automation_internal/automation_internal_api.cc

Index: chrome/browser/extensions/api/automation_internal/automation_internal_api.cc
diff --git a/chrome/browser/extensions/api/automation_internal/automation_internal_api.cc b/chrome/browser/extensions/api/automation_internal/automation_internal_api.cc
index 40e184bbf3b64d5776b523af8abb94ca9feb063d..9df8a42e62ca87f6980c11c5e216987e2ddd5fa3 100644
--- a/chrome/browser/extensions/api/automation_internal/automation_internal_api.cc
+++ b/chrome/browser/extensions/api/automation_internal/automation_internal_api.cc
@@ -16,6 +16,7 @@
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/common/extensions/api/automation_internal.h"
 #include "chrome/common/extensions/manifest_handlers/automation.h"
+#include "content/public/browser/browser_accessibility_state.h"
 #include "content/public/browser/ax_event_notification_details.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/render_process_host.h"
@@ -167,7 +168,26 @@ AutomationInternalEnableTabFunction::Run() {
   return RespondNow(
       ArgumentList(api::automation_internal::EnableTab::Results::Create(
           rfh->GetProcess()->GetID(), rfh->GetRoutingID())));
-  }
+}
+
+ExtensionFunction::ResponseAction
+AutomationInternalEnableRendererFunction::Run() {
+  using api::automation_internal::EnableRenderer::Params;
+  scoped_ptr<Params> params(Params::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());
+  content::RenderFrameHost* rfh =
+      content::RenderFrameHost::FromID(params->process_id,

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

I think this is what we want eventually, but aren't the routing ids used in the
automation id right now RenderView routing ids and not RenderFrame ids? It seems
confusing to mix them.

+                                       params->routing_id);
+  if (!rfh)
+    return RespondNow(Error("unable to load tab"));
+
+ content::WebContents* contents =

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

nit: indent

+        content::WebContents::FromRenderFrameHost(rfh);
+    AutomationWebContentsObserver::CreateForWebContents(contents);
+    contents->EnableTreeOnlyAccessibilityMode();
+
+  return RespondNow(NoArguments());
+}
 
 ExtensionFunction::ResponseAction
 AutomationInternalPerformActionFunction::Run() {