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

chrome/renderer/devtools_agent.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/renderer/devtools_agent.h"
 
 #include <map>
 
 #include "base/command_line.h"
 #include "chrome/common/chrome_switches.h"
@@ skipping 35 matching lines @@
   }
  private:
   MessageLoop* message_loop_;
 };
 
 } //  namespace
 
 // 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) {
-  agent_for_routing_id_[routing_id] = this;
+DevToolsAgent::DevToolsAgent(RenderView* render_view)
+    : RenderViewObserver(render_view) {
+  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_);
-}
-
-void DevToolsAgent::OnNavigate() {
-  WebDevToolsAgent* web_agent = GetWebAgent();
-  if (web_agent) {
-    web_agent->didNavigate();
-  }
+  agent_for_routing_id_.erase(routing_id());
 }
 
 // Called on the Renderer thread.
 bool DevToolsAgent::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(DevToolsAgent, message)
     IPC_MESSAGE_HANDLER(DevToolsAgentMsg_Attach, OnAttach)
     IPC_MESSAGE_HANDLER(DevToolsAgentMsg_Detach, OnDetach)
     IPC_MESSAGE_HANDLER(DevToolsAgentMsg_FrontendLoaded, OnFrontendLoaded)
     IPC_MESSAGE_HANDLER(DevToolsAgentMsg_DispatchOnInspectorBackend,
                         OnDispatchOnInspectorBackend)
     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"));
 }
 
 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()));
 }
 
 WebCString DevToolsAgent::debuggerScriptSource() {
   base::StringPiece debuggerScriptjs =
       webkit_glue::GetDataResource(IDR_DEVTOOLS_DEBUGGER_SCRIPT_JS);
   return WebCString(debuggerScriptjs.data(), debuggerScriptjs.length());
 }
 
@@ skipping 50 matching lines @@
 }
 
 void DevToolsAgent::OnInspectElement(int x, int y) {
   WebDevToolsAgent* web_agent = GetWebAgent();
   if (web_agent) {
     web_agent->attach();
     web_agent->inspectElementAt(WebPoint(x, y));
   }
 }
 
+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();
 }