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

Unified Diff: chrome/browser/automation/testing_automation_provider.cc

Issue 10257015: Added a new automation hook ExecuteJavascriptInOOBEWebUI() to execute javascript prior to login on… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Ready for review. Created 8 years, 8 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/automation/testing_automation_provider.cc
diff --git a/chrome/browser/automation/testing_automation_provider.cc b/chrome/browser/automation/testing_automation_provider.cc
index 09b66736b8342b61d42d3f83977ab601cbb3ca51..5a9088d607859b86ddf5d4727343a07083abc2e0 100644
--- a/chrome/browser/automation/testing_automation_provider.cc
+++ b/chrome/browser/automation/testing_automation_provider.cc
@@ -1312,9 +1312,7 @@ void TestingAutomationProvider::GetFullscreenBubbleVisibility(int handle,
}
}
-namespace {
-
-void ExecuteJavascriptInRenderViewFrame(
+void TestingAutomationProvider::ExecuteJavascriptInRenderViewFrame(
const string16& frame_xpath,
const string16& script,
IPC::Message* reply_message,
@@ -1330,8 +1328,6 @@ void ExecuteJavascriptInRenderViewFrame(
frame_xpath, script);
}
-} // namespace
-
void TestingAutomationProvider::ExecuteJavascript(
int handle,
const std::wstring& frame_xpath,
@@ -2172,6 +2168,8 @@ void TestingAutomationProvider::SendJSONRequest(int handle,
handler_map["SetPrefs"] = &TestingAutomationProvider::SetPrefs;
handler_map["ExecuteJavascript"] =
&TestingAutomationProvider::ExecuteJavascriptJSON;
+ handler_map["ExecuteJavascriptInLogin"] =
Nirnimesh 2012/04/27 22:46:38 move to OS_CHROMEOS section below
craigdh 2012/04/27 23:29:50 Done.
+ &TestingAutomationProvider::ExecuteJavascriptInLogin;
handler_map["AddDomEventObserver"] =
&TestingAutomationProvider::AddDomEventObserver;
handler_map["RemoveEventObserver"] =

Powered by Google App Engine
This is Rietveld 408576698