[DevTools] Expose DevToolsAgentHost for OOPIF, display it in chrome://inspect.

content/browser/devtools/render_frame_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/devtools/render_frame_devtools_agent_host.h"
 
 #include "base/basictypes.h"
 #include "base/json/json_writer.h"
 #include "base/lazy_instance.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 62 matching lines @@
 
 scoped_refptr<DevToolsAgentHost>
 DevToolsAgentHost::GetOrCreateFor(WebContents* web_contents) {
   RenderFrameDevToolsAgentHost* result = FindAgentHost(web_contents);
   if (!result)
     result = new RenderFrameDevToolsAgentHost(web_contents->GetMainFrame());
   return result;
 }
 
 // static
+scoped_refptr<DevToolsAgentHost> RenderFrameDevToolsAgentHost::GetOrCreateFor(
+    RenderFrameHost* host) {
+  RenderFrameDevToolsAgentHost* result = FindAgentHost(host);
+  if (!result)
+    result = new RenderFrameDevToolsAgentHost(host);
+  return result;
+}
+
+// static
+void RenderFrameDevToolsAgentHost::AppendAgentHostForFrameIfApplicable(
+    std::vector<scoped_refptr<DevToolsAgentHost>>* result,
+    RenderFrameHost* host) {
+  RenderFrameHostImpl* rfh = static_cast<RenderFrameHostImpl*>(host);
+  if (rfh->IsRenderFrameLive() && rfh->IsFrameGroupRoot())
+    result->push_back(RenderFrameDevToolsAgentHost::GetOrCreateFor(rfh));
+}
+
+// static
 bool DevToolsAgentHost::HasFor(WebContents* web_contents) {
   return FindAgentHost(web_contents) != NULL;
 }
 
 // static
 bool DevToolsAgentHost::IsDebuggerAttached(WebContents* web_contents) {
   RenderFrameDevToolsAgentHost* agent_host = FindAgentHost(web_contents);
   return agent_host && agent_host->IsAttached();
 }
 
 //static
-std::vector<WebContents*> DevToolsAgentHostImpl::GetInspectableWebContents() {
-  std::set<WebContents*> set;
-  scoped_ptr<RenderWidgetHostIterator> widgets(
-      RenderWidgetHost::GetRenderWidgetHosts());
-  while (RenderWidgetHost* widget = widgets->GetNextHost()) {
-    // Ignore processes that don't have a connection, such as crashed contents.
-    if (!widget->GetProcess()->HasConnection())
-      continue;
-    if (!widget->IsRenderView())
-      continue;
-
-    RenderViewHost* rvh = RenderViewHost::From(widget);
-    WebContents* web_contents = WebContents::FromRenderViewHost(rvh);
-    if (web_contents)
-      set.insert(web_contents);
-  }
-  std::vector<WebContents*> result(set.size());
-  std::copy(set.begin(), set.end(), result.begin());
-  return result;
+void RenderFrameDevToolsAgentHost::AddAllAgentHosts(
+    std::vector<scoped_refptr<DevToolsAgentHost>>* result) {
+  base::Callback<void(RenderFrameHost*)> callback = base::Bind(
+      RenderFrameDevToolsAgentHost::AppendAgentHostForFrameIfApplicable,
+      base::Unretained(result));
+  for (const auto& wc : WebContentsImpl::GetAllWebContents())
+    wc->ForEachFrame(callback);
 }
 
 // static
 void RenderFrameDevToolsAgentHost::OnCancelPendingNavigation(
     RenderFrameHost* pending,
     RenderFrameHost* current) {
   RenderFrameDevToolsAgentHost* agent_host = FindAgentHost(pending);
   if (!agent_host)
     return;
   agent_host->DisconnectRenderFrameHost();
@@ skipping 313 matching lines @@
 
 void RenderFrameDevToolsAgentHost::DisconnectWebContents() {
   DisconnectRenderFrameHost();
 }
 
 void RenderFrameDevToolsAgentHost::ConnectWebContents(WebContents* wc) {
   ConnectRenderFrameHost(wc->GetMainFrame());
 }
 
 DevToolsAgentHost::Type RenderFrameDevToolsAgentHost::GetType() {
-  return TYPE_WEB_CONTENTS;
+  return IsChildFrame() ? TYPE_FRAME : TYPE_WEB_CONTENTS;
 }
 
 std::string RenderFrameDevToolsAgentHost::GetTitle() {
+  if (IsChildFrame())
+    return GetURL().spec();
   if (WebContents* web_contents = GetWebContents())
     return base::UTF16ToUTF8(web_contents->GetTitle());
   return "";
 }
 
 GURL RenderFrameDevToolsAgentHost::GetURL() {
+  if (IsChildFrame())
+    return render_frame_host_->GetLastCommittedURL();

[nasko, 2015/03/05 15:20:48]

Why not return this in all cases?

[dgozman, 2015/03/06 16:14:14]

Done.

   if (WebContents* web_contents = GetWebContents())
     return web_contents->GetVisibleURL();

[nasko, 2015/03/05 15:20:48]

This is a bit orthogonal, but why do you need the visible URL? In my mind,
DevTools always operates on a document that exists. The visible URL can be an
URL that isn't committed yet, so no real document exist associated with it.

[dgozman, 2015/03/06 16:14:14]

You are right, using last committed url.

   return render_frame_host_ ?
       render_frame_host_->GetLastCommittedURL() : GURL();
 }
 
 bool RenderFrameDevToolsAgentHost::Activate() {
   if (render_frame_host_) {
     render_frame_host_->GetRenderViewHost()->GetDelegate()->Activate();
     return true;
   }
   return false;
@@ skipping 54 matching lines @@
   ProcessChunkedMessageFromAgent(message);
 }
 
 void RenderFrameDevToolsAgentHost::DispatchOnInspectorFrontend(
     const std::string& message) {
   if (!IsAttached() || !render_frame_host_)
     return;
   SendMessageToClient(message);
 }
 
+bool RenderFrameDevToolsAgentHost::IsChildFrame() {
+  return render_frame_host_ && render_frame_host_->GetParent();
+}
+
 }  // namespace content