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

content/browser/debugger/devtools_agent_host.h

Index: content/browser/debugger/devtools_agent_host.h
diff --git a/content/browser/debugger/devtools_agent_host.h b/content/browser/debugger/devtools_agent_host.h
index b4b64087955a307a5b3f9e4ca84a9eea5beb575e..128ee6e885568a555a743568f43dac3ee9a8933e 100644
--- a/content/browser/debugger/devtools_agent_host.h
+++ b/content/browser/debugger/devtools_agent_host.h
@@ -35,6 +35,7 @@ class CONTENT_EXPORT DevToolsAgentHost {
   void AddMessageToConsole(ConsoleMessageLevel level,
                            const std::string& message);
 
+  virtual void NotifyClientOpening() = 0;

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

These two notification methods should be protected or private now that they are
only called from DevToolsAgentHost. I'd probably rename these methods to
NotifyClientAttachingDetaching.

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

Done.

   // TODO(yurys): get rid of this method

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

Please remove this comment.

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

Done.

   virtual void NotifyClientClosing() = 0;