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

content/public/browser/notification_types.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_NOTIFICATION_TYPES_H_
 #define CONTENT_PUBLIC_BROWSER_NOTIFICATION_TYPES_H_
 
 // This file describes various types used to describe and filter notifications
 // that pass through the NotificationService.
 //
@@ skipping 145 matching lines @@
 
   // This message is sent when the application is terminating (the last
   // browser window has shutdown as part of an explicit user-initiated exit,
   // or the user closed the last browser window on Windows/Linux and there are
   // no BackgroundContents keeping the browser running). No source or details
   // are passed.
   NOTIFICATION_APP_TERMINATING,
 
   // Devtools ------------------------------------------------------------------
 
   // Indicates that a devtools window is opening. The source is the

[yurys, 2012/08/21 19:52:55]

Please update the comment. At least type of the details object has changed.

[yurys, 2012/08/21 20:08:55]

I was wrong, it's still RVH. Ignore this comment.

[Matt Perry, 2012/08/21 20:13:46]

Updated comment. But the details are still the RVH.

   // BrowserContext* and the details is the inspected RenderViewHost*.
-  NOTIFICATION_DEVTOOLS_WINDOW_OPENING,
+  NOTIFICATION_DEVTOOLS_AGENT_ATTACHED,
 
   // Indicates that a devtools window is closing. The source is the
   // BrowserContext* and the details is the inspected RenderViewHost*.
-  NOTIFICATION_DEVTOOLS_WINDOW_CLOSING,
+  NOTIFICATION_DEVTOOLS_AGENT_DETACHED,
 
   // WebContents ---------------------------------------------------------------
 
   // This notification is sent when a render view host has connected to a
   // renderer process. The source is a Source<WebContents> with a pointer to
   // the WebContents.  A WEB_CONTENTS_DISCONNECTED notification is
   // guaranteed before the source pointer becomes junk.  No details are
   // expected.
   NOTIFICATION_WEB_CONTENTS_CONNECTED,
 
@@ skipping 185 matching lines @@
   // of a temporary zoom level change, the details is an empty string.
   NOTIFICATION_ZOOM_LEVEL_CHANGED,
 
   // Custom notifications used by the embedder should start from here.
   NOTIFICATION_CONTENT_END,
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_NOTIFICATION_TYPES_H_