Commit fe9cc5eb authored by lukasza's avatar lukasza Committed by Commit Bot

Remove all calls to domAutomationController.setAutomationId.

These calls are no longer needed after r485461
(https://crrev.com/2478803003) which has made the
domAutomationController.setAutomationId call a no-op.

BUG=662543
This CL was uploaded by git cl split.

R=tommi@chromium.org

Review-Url: https://codereview.chromium.org/2977633004
Cr-Commit-Position: refs/heads/master@{#487944}
parent d27fb386
......@@ -137,17 +137,13 @@ bool ExecuteScriptHelper(RenderFrameHost* render_frame_host,
std::unique_ptr<base::Value>* result)
WARN_UNUSED_RESULT;
// Executes the passed |original_script| in the frame specified by
// |render_frame_host|. If |result| is not NULL, stores the value that the
// evaluation of the script in |result|. Returns true on success.
// Executes the passed |script| in the frame specified by |render_frame_host|.
// If |result| is not NULL, stores the value that the evaluation of the script
// in |result|. Returns true on success.
bool ExecuteScriptHelper(RenderFrameHost* render_frame_host,
const std::string& original_script,
const std::string& script,
bool user_gesture,
std::unique_ptr<base::Value>* result) {
// TODO(jcampan): we should make the domAutomationController not require an
// automation id.
std::string script =
"window.domAutomationController.setAutomationId(0);" + original_script;
// TODO(lukasza): Only get messages from the specific |render_frame_host|.
DOMMessageQueue dom_message_queue(
WebContents::FromRenderFrameHost(render_frame_host));
......@@ -216,12 +212,8 @@ bool ExecuteScriptInIsolatedWorldHelper(RenderFrameHost* render_frame_host,
bool ExecuteScriptInIsolatedWorldHelper(RenderFrameHost* render_frame_host,
const int world_id,
const std::string& original_script,
const std::string& script,
std::unique_ptr<base::Value>* result) {
// TODO(jcampan): we should make the domAutomationController not require an
// automation id.
std::string script =
"window.domAutomationController.setAutomationId(0);" + original_script;
// TODO(lukasza): Only get messages from the specific |render_frame_host|.
DOMMessageQueue dom_message_queue(
WebContents::FromRenderFrameHost(render_frame_host));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment