Remove DOMAutomationController::automation_id_

content/public/test/browser_test_utils.h

Index: content/public/test/browser_test_utils.h
diff --git a/content/public/test/browser_test_utils.h b/content/public/test/browser_test_utils.h
index da139cd59f7e811f1a14b406cd613931cde99291..0a187f104160bccb767fa008b0e5bc0aad222470 100644
--- a/content/public/test/browser_test_utils.h
+++ b/content/public/test/browser_test_utils.h
@@ -219,11 +219,18 @@ bool ExecuteScriptWithoutUserGesture(const ToRenderFrameHost& adapter,
                                      const std::string& script)
     WARN_UNUSED_RESULT;
 
-// The following methods execute the passed |script| in the specified frame with
-// the user gesture and set |result| to the value passed to
-// "window.domAutomationController.send" by the executed script. They return
-// true on success, false if the script execution failed or did not evaluate to
-// the expected type.
+// Kicks off execution of the |script| in the specified frame and returns
+// immediately (unlike ExecuteScript above).

[mmenke, 2017/07/06 16:10:34]

I guess ExecuteScript waits for completion?  That should be documented with
ExecuteScript.

[Łukasz Anforowicz, 2017/07/06 16:46:56]

Done.

+//
+// This function is useful when extra calls to domAutomationController.send
+// are undesirable.
+void ExecuteUnmodifiedScript(const ToRenderFrameHost& adapter,

[mmenke, 2017/07/06 16:10:34]

ExecuteUnmodifiedScript?  Does ExecuteScript modify the script?  If so, perhaps
that should be documented, as it's not clear what makes this script not
modified.

[Łukasz Anforowicz, 2017/07/06 16:46:56]

Done.

+                             const std::string& script);
+
+// The following methods execute the passed |script| in the specified frame and
+// sets |result| to the value passed to "window.domAutomationController.send" by
+// the executed script. They return true on success, false if the script
+// execution failed or did not evaluate to the expected type.
 bool ExecuteScriptAndExtractDouble(const ToRenderFrameHost& adapter,
                                    const std::string& script,
                                    double* result) WARN_UNUSED_RESULT;
@@ -521,6 +528,7 @@ class DOMMessageQueue : public NotificationObserver,
   NotificationRegistrar registrar_;
   std::queue<std::string> message_queue_;
   scoped_refptr<MessageLoopRunner> message_loop_runner_;
+  bool renderer_crashed_ = false;
 
   DISALLOW_COPY_AND_ASSIGN(DOMMessageQueue);
 };