Modifying extension automation so that it is done through a particular

tab for all extension views.  Previously, API routing to the
automation client would only work for extension views that were
contained in the particular ExternalTab instance being automated. This
meant you couldn't test API calls from e.g. background pages.

Also using async functions instead of the previous RVH-based hack.

Updating one of the UI tests to make the API calls from a background
page, to provide an end-to-end test of the new routing.

This makes AutomationProvider::SetEnableAutomationExtension
Windows-only, but it effectively always was since it was already
dependent on ExternalTabContainer (indirectly, to provide a non-empty
implementation of TabContentsDelegate::ForwardMessageToExternalHost).

BUG=none
TEST=ui_tests.exe, chrome_frame_tests.exe, chrome_frame_unittests.exe

Review URL: http://codereview.chromium.org/366025

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31403 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 15 insertions, 0 deletions

diff --git a/chrome/browser/automation/automation_provider_win.cc b/chrome/browser/automation/automation_provider_win.cc
index cfadf91..7b750ca 100644
--- a/chrome/browser/automation/automation_provider_win.cc
+++ b/chrome/browser/automation/automation_provider_win.cc
@@ -489,3 +489,18 @@ void AutomationProvider::TerminateSession(int handle, bool* success) {
     *success = (::PostMessageW(window, WM_ENDSESSION, 0, 0) == TRUE);
   }
 }
+
+void AutomationProvider::SetEnableExtensionAutomation(
+    int tab_handle,
+    const std::vector<std::string>& functions_enabled) {
+  ExternalTabContainer* external_tab = GetExternalTabForHandle(tab_handle);
+  if (external_tab) {
+    external_tab->SetEnableExtensionAutomation(functions_enabled);
+  } else {
+    // Tab must exist, and must be an external tab so that its
+    // delegate has an on-empty
+    // implementation of ForwardMessageToExternalHost.
+    DLOG(WARNING) <<
+      "SetEnableExtensionAutomation called with invalid tab handle.";
+  }
+}