Introduce RenderView::Observer interface so that RenderView doesn't have to k...

chrome/renderer/devtools_agent.cc

Index: chrome/renderer/devtools_agent.cc
===================================================================
--- chrome/renderer/devtools_agent.cc	(revision 71253)
+++ chrome/renderer/devtools_agent.cc	(working copy)
@@ -10,7 +10,6 @@
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/renderer/devtools_agent_filter.h"
-#include "chrome/renderer/render_view.h"
 #include "grit/webkit_chromium_resources.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebDevToolsAgent.h"
 #include "third_party/WebKit/WebKit/chromium/public/WebPoint.h"
@@ -53,26 +52,17 @@
 // static
 std::map<int, DevToolsAgent*> DevToolsAgent::agent_for_routing_id_;
 
-DevToolsAgent::DevToolsAgent(int routing_id, RenderView* render_view)
-    : routing_id_(routing_id),
-      render_view_(render_view) {
+DevToolsAgent::DevToolsAgent(int routing_id) {
   agent_for_routing_id_[routing_id] = this;
 
   CommandLine* cmd = CommandLine::ForCurrentProcess();
-  expose_v8_debugger_protocol_ =cmd->HasSwitch(switches::kRemoteShellPort);
+  expose_v8_debugger_protocol_ = cmd->HasSwitch(switches::kRemoteShellPort);
 }
 
 DevToolsAgent::~DevToolsAgent() {
-  agent_for_routing_id_.erase(routing_id_);
+  agent_for_routing_id_.erase(routing_id());
 }
 
-void DevToolsAgent::OnNavigate() {
-  WebDevToolsAgent* web_agent = GetWebAgent();
-  if (web_agent) {
-    web_agent->didNavigate();
-  }
-}
-
 // Called on the Renderer thread.
 bool DevToolsAgent::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
@@ -85,33 +75,35 @@
     IPC_MESSAGE_HANDLER(DevToolsAgentMsg_InspectElement, OnInspectElement)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
+
+  if (message.type() == ViewMsg_Navigate::ID)
+    OnNavigate();  // Don't want to swallow the message.
+
   return handled;
 }
 
 void DevToolsAgent::sendMessageToInspectorFrontend(
     const WebKit::WebString& message) {
-  IPC::Message* m = new ViewHostMsg_ForwardToDevToolsClient(
-      routing_id_,
-      DevToolsClientMsg_DispatchOnInspectorFrontend(message.utf8()));
-  render_view_->Send(m);
+  Send(new ViewHostMsg_ForwardToDevToolsClient(
+      routing_id(),
+      DevToolsClientMsg_DispatchOnInspectorFrontend(message.utf8())));
 }
 
 void DevToolsAgent::sendDebuggerOutput(const WebKit::WebString& data) {
-  IPC::Message* m = new ViewHostMsg_ForwardToDevToolsClient(
-      routing_id_,
-      DevToolsClientMsg_DebuggerOutput(data.utf8()));
-  render_view_->Send(m);
+  Send(new ViewHostMsg_ForwardToDevToolsClient(
+      routing_id(),
+      DevToolsClientMsg_DebuggerOutput(data.utf8())));
 }
 
 int DevToolsAgent::hostIdentifier() {
-  return routing_id_;
+  return routing_id();
 }
 
 void DevToolsAgent::runtimeFeatureStateChanged(
     const WebKit::WebString& feature,
     bool enabled) {
-  render_view_->Send(new ViewHostMsg_DevToolsRuntimePropertyChanged(
-      routing_id_,
+  Send(new ViewHostMsg_DevToolsRuntimePropertyChanged(
+      routing_id(),
       feature.utf8(),
       enabled ? "true" : "false"));
 }
@@ -119,8 +111,8 @@
 void DevToolsAgent::runtimePropertyChanged(
     const WebKit::WebString& name,
     const WebKit::WebString& value) {
-  render_view_->Send(new ViewHostMsg_DevToolsRuntimePropertyChanged(
-      routing_id_,
+  Send(new ViewHostMsg_DevToolsRuntimePropertyChanged(
+      routing_id(),
       name.utf8(),
       value.utf8()));
 }
@@ -191,8 +183,15 @@
   }
 }
 
+void DevToolsAgent::OnNavigate() {
+  WebDevToolsAgent* web_agent = GetWebAgent();
+  if (web_agent) {
+    web_agent->didNavigate();
+  }
+}
+
 WebDevToolsAgent* DevToolsAgent::GetWebAgent() {
-  WebView* web_view = render_view_->webview();
+  WebView* web_view = render_view()->webview();
   if (!web_view)
     return NULL;
   return web_view->devToolsAgent();