Switch DomAutomationController to be a RenderFrameObserver.

content/browser/frame_host/render_frame_host_impl.cc

Index: content/browser/frame_host/render_frame_host_impl.cc
diff --git a/content/browser/frame_host/render_frame_host_impl.cc b/content/browser/frame_host/render_frame_host_impl.cc
index 304bcaf8b11ced008b8c49e2dcc039fb548dabac..3e49aad1faf207fcdca35d7b53887ad94c048182 100644
--- a/content/browser/frame_host/render_frame_host_impl.cc
+++ b/content/browser/frame_host/render_frame_host_impl.cc
@@ -16,6 +16,7 @@
 #include "content/common/frame_messages.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/content_browser_client.h"
+#include "content/public/browser/dom_operation_notification_details.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "content/public/browser/user_metrics.h"
@@ -144,6 +145,8 @@ bool RenderFrameHostImpl::OnMessageReceived(const IPC::Message &msg) {
     IPC_MESSAGE_HANDLER(FrameHostMsg_DidStopLoading, OnDidStopLoading)
     IPC_MESSAGE_HANDLER(FrameHostMsg_SwapOut_ACK, OnSwapOutACK)
     IPC_MESSAGE_HANDLER(FrameHostMsg_ContextMenu, OnContextMenu)
+    IPC_MESSAGE_HANDLER(FrameHostMsg_DomOperationResponse,
+                        OnDomOperationResponse)
   IPC_END_MESSAGE_MAP_EX()
 
   if (!msg_is_ok) {
@@ -347,6 +350,12 @@ void RenderFrameHostImpl::OnContextMenu(const ContextMenuParams& params) {
   delegate_->ShowContextMenu(this, validated_params);
 }
 
+void RenderFrameHostImpl::OnDomOperationResponse(const std::string& json_string,

[Charlie Reis, 2014/02/15 01:47:33]

Do we need to handle this message here?  Maybe we can just handle it in
WebContentsImpl (and InterstitialPageImpl) if all we're doing is passing it to
the delegate.  Then it doesn't need to be on RenderFrameHostDelegate either.

[mkosiba (inactive), 2014/02/17 17:42:31]

yes, that does indeed make more sense.

+                                                 int automation_id) {
+  delegate_->OnDomOperationResponse(
+      DomOperationNotificationDetails(json_string, automation_id));
+}
+
 void RenderFrameHostImpl::SetPendingShutdown(const base::Closure& on_swap_out) {
   render_view_host_->SetPendingShutdown(on_swap_out);
 }