[DevTools] Implement Inspector.interstitial events.

content/browser/devtools/render_view_devtools_agent_host.cc

Index: content/browser/devtools/render_view_devtools_agent_host.cc
diff --git a/content/browser/devtools/render_view_devtools_agent_host.cc b/content/browser/devtools/render_view_devtools_agent_host.cc
index 87de83197993cf69f1756d904e0b5b5d94ecca84..c1f7d1ffa1ba81fc806e6178a94aca53deac976a 100644
--- a/content/browser/devtools/render_view_devtools_agent_host.cc
+++ b/content/browser/devtools/render_view_devtools_agent_host.cc
@@ -319,6 +319,8 @@ bool RenderViewDevToolsAgentHost::OnMessageReceived(
 }
 
 void RenderViewDevToolsAgentHost::DidAttachInterstitialPage() {
+  overrides_handler_->DidAttachInterstitialPage();

[pfeldman, 2014/08/22 13:08:46]

One of the main protocol rules is that the domain is silent unless user requests
something. You should only issue this in case Inspector.enable (or Page.enable)
has been called depending on the domain notification belongs. I wonder if we
should move it to Page.* so that the client does not need to enable the
inspector domain just for that.

[dgozman, 2014/08/22 16:29:28]

Done.

+
   if (!render_view_host_)
     return;
   // The rvh set in AboutToNavigateRenderView turned out to be interstitial.
@@ -331,6 +333,10 @@ void RenderViewDevToolsAgentHost::DidAttachInterstitialPage() {
   ConnectRenderViewHost(web_contents->GetRenderViewHost());
 }
 
+void RenderViewDevToolsAgentHost::DidDetachInterstitialPage() {
+  overrides_handler_->DidDetachInterstitialPage();
+}
+
 void RenderViewDevToolsAgentHost::Observe(int type,
                                           const NotificationSource& source,
                                           const NotificationDetails& details) {