Fix a bug where the inspector would not keep event pages alive.

content/browser/debugger/render_view_devtools_agent_host.cc

 // Copyright (c) 2012 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 "content/browser/debugger/render_view_devtools_agent_host.h"
 
 #include "base/basictypes.h"
 #include "base/lazy_instance.h"
 #include "content/browser/debugger/devtools_manager_impl.h"
 #include "content/browser/debugger/render_view_devtools_agent_host.h"
@@ skipping 59 matching lines @@
       return true;
   }
   return false;
 }
 
 RenderViewDevToolsAgentHost::RenderViewDevToolsAgentHost(
     RenderViewHost* rvh)
     : content::RenderViewHostObserver(rvh),
       render_view_host_(rvh) {
   g_instances.Get()[rvh] = this;
-
-  // Notify that the view is being opened. This allows any views being debugged
-  // to do anything special they need to do to support debugging.
-  content::NotificationService::current()->Notify(
-      content::NOTIFICATION_DEVTOOLS_WINDOW_OPENING,
-      content::Source<content::BrowserContext>(
-          render_view_host_->GetSiteInstance()->GetProcess()->
-              GetBrowserContext()),
-      content::Details<RenderViewHost>(render_view_host_));
 }
 
 void RenderViewDevToolsAgentHost::SendMessageToAgent(IPC::Message* msg) {
   msg->set_routing_id(render_view_host_->GetRoutingID());
   render_view_host_->Send(msg);
 }
 
-void RenderViewDevToolsAgentHost::NotifyClientClosing() {
+void RenderViewDevToolsAgentHost::NotifyClientAttaching() {
   content::NotificationService::current()->Notify(
-      content::NOTIFICATION_DEVTOOLS_WINDOW_CLOSING,
+      content::NOTIFICATION_DEVTOOLS_AGENT_ATTACHED,
       content::Source<content::BrowserContext>(
           render_view_host_->GetSiteInstance()->GetProcess()->
               GetBrowserContext()),
+      content::Details<RenderViewHost>(render_view_host_));
+}
+
+void RenderViewDevToolsAgentHost::NotifyClientDetaching() {
+  content::NotificationService::current()->Notify(
+      content::NOTIFICATION_DEVTOOLS_AGENT_DETACHED,
+      content::Source<content::BrowserContext>(
+          render_view_host_->GetSiteInstance()->GetProcess()->
+              GetBrowserContext()),
       content::Details<RenderViewHost>(render_view_host_));
 }
 
 int RenderViewDevToolsAgentHost::GetRenderProcessId() {
   return render_view_host_->GetProcess()->GetID();
 }
 
 RenderViewDevToolsAgentHost::~RenderViewDevToolsAgentHost() {
   g_instances.Get().erase(render_view_host_);
 }
@@ skipping 33 matching lines @@
 
 void RenderViewDevToolsAgentHost::OnClearBrowserCache() {
   content::GetContentClient()->browser()->ClearCache(render_view_host_);
 }
 
 void RenderViewDevToolsAgentHost::OnClearBrowserCookies() {
   content::GetContentClient()->browser()->ClearCookies(render_view_host_);
 }
 
 }  // namespace content