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/17 07:05:01]

Is there any reason for not sending this notification from
DevToolsAgentHost::Attach? That way it would work for remote debugging as well,
not only for the in browser devtools.

[Matt Perry, 2012/08/17 23:11:46]

I'm not really familiar with the devtools code. If you think that's a better
place to put it, I can move it there. Should I also move NotifyClientClosing()
to DevToolsAgentHost::Detach?

To be clear, these notifications are sent to indicate that a devtools UI window
has opened/closed.

   // TODO(yurys): get rid of this method
   virtual void NotifyClientClosing() = 0;