Revert of DevTools: merge devtools target with devtools host, part 1

content/browser/devtools/render_frame_devtools_agent_host.cc

Index: content/browser/devtools/render_frame_devtools_agent_host.cc
diff --git a/content/browser/devtools/render_frame_devtools_agent_host.cc b/content/browser/devtools/render_frame_devtools_agent_host.cc
index 7893034ce695737ef2b32e8c351420bce0a162e0..c3353de2d8600b2fae66b7c54b0b03b84daa4515 100644
--- a/content/browser/devtools/render_frame_devtools_agent_host.cc
+++ b/content/browser/devtools/render_frame_devtools_agent_host.cc
@@ -13,7 +13,6 @@
 #include "content/browser/bad_message.h"
 #include "content/browser/child_process_security_policy_impl.h"
 #include "content/browser/devtools/devtools_frame_trace_recorder.h"
-#include "content/browser/devtools/devtools_manager.h"
 #include "content/browser/devtools/devtools_protocol_handler.h"
 #include "content/browser/devtools/page_navigation_throttle.h"
 #include "content/browser/devtools/protocol/browser_handler.h"
@@ -858,45 +857,15 @@
   WebContentsObserver::Observe(WebContents::FromRenderFrameHost(host));
 }
 
-std::string RenderFrameDevToolsAgentHost::GetParentId() {
-  if (IsChildFrame()) {
-    RenderFrameHostImpl* frame_host = current_->host();
-    while (frame_host && !ShouldCreateDevToolsFor(frame_host))
-      frame_host = frame_host->GetParent();
-    if (frame_host)
-      return DevToolsAgentHost::GetOrCreateFor(frame_host)->GetId();
-  }
-
-  WebContentsImpl* contents = static_cast<WebContentsImpl*>(web_contents());
-  if (!contents)
-    return "";
-  contents = contents->GetOuterWebContents();
-  if (contents)
-    return DevToolsAgentHost::GetOrCreateFor(contents)->GetId();
-  return "";
-}
-
-std::string RenderFrameDevToolsAgentHost::GetType() {
-  DevToolsManager* manager = DevToolsManager::GetInstance();
-  if (manager->delegate())
-    return manager->delegate()->GetTargetType(current_->host());
-  return kTypeOther;
+DevToolsAgentHost::Type RenderFrameDevToolsAgentHost::GetType() {
+  return IsChildFrame() ? TYPE_FRAME : TYPE_WEB_CONTENTS;
 }
 
 std::string RenderFrameDevToolsAgentHost::GetTitle() {
-  DevToolsManager* manager = DevToolsManager::GetInstance();
-  std::string result;
-  if (manager->delegate())
-    result = manager->delegate()->GetTargetTitle(current_->host());
-  if (!result.empty())
-    return result;
-  content::WebContents* web_contents = GetWebContents();
-  if (web_contents)
-    result = base::UTF16ToUTF8(web_contents->GetTitle());
-  return GetURL().spec();
-}
-
-std::string RenderFrameDevToolsAgentHost::GetDescription() {
+  if (IsChildFrame())
+    return GetURL().spec();
+  if (WebContents* web_contents = GetWebContents())
+    return base::UTF16ToUTF8(web_contents->GetTitle());
   return "";
 }
 
@@ -912,10 +881,6 @@
   return GURL();
 }
 
-GURL RenderFrameDevToolsAgentHost::GetFaviconURL() {
-  return GURL();
-}
-
 bool RenderFrameDevToolsAgentHost::Activate() {
   WebContentsImpl* wc = static_cast<WebContentsImpl*>(web_contents());
   if (wc) {
@@ -925,24 +890,12 @@
   return false;
 }
 
-void RenderFrameDevToolsAgentHost::Reload() {
-  WebContentsImpl* wc = static_cast<WebContentsImpl*>(web_contents());
-  if (wc)
-    wc->GetController().Reload(true);
-}
-
 bool RenderFrameDevToolsAgentHost::Close() {
   if (web_contents()) {
     web_contents()->ClosePage();
     return true;
   }
   return false;
-}
-
-base::TimeTicks RenderFrameDevToolsAgentHost::GetLastActivityTime() {
-  if (content::WebContents* contents = web_contents())
-    return contents->GetLastActiveTime();
-  return base::TimeTicks();
 }
 
 void RenderFrameDevToolsAgentHost::OnSwapCompositorFrame(
@@ -999,7 +952,8 @@
   if (IsAttached() && sender->GetRoutingID() != new_routing_id && frame_host) {
     scoped_refptr<DevToolsAgentHost> agent =
         DevToolsAgentHost::GetOrCreateFor(frame_host);
-    success = agent->Inspect();
+    success = static_cast<DevToolsAgentHostImpl*>(agent.get())->
+        Inspect(agent->GetBrowserContext());
   }
 
   sender->Send(new DevToolsAgentMsg_RequestNewWindow_ACK(