Extract debugger target enumeration into a separate class

content/browser/devtools/devtools_agent_host_impl.cc

Index: content/browser/devtools/devtools_agent_host_impl.cc
diff --git a/content/browser/devtools/devtools_agent_host_impl.cc b/content/browser/devtools/devtools_agent_host_impl.cc
index 0474103a4232e1c39c790fda2d4acd26b2442466..94e129636401ee78ae6d94e358a82ecbade743b5 100644
--- a/content/browser/devtools/devtools_agent_host_impl.cc
+++ b/content/browser/devtools/devtools_agent_host_impl.cc
@@ -5,17 +5,24 @@
 #include "content/browser/devtools/devtools_agent_host_impl.h"
 
 #include "base/basictypes.h"
+#include "base/stringprintf.h"
 #include "content/common/devtools_messages.h"
+#include "content/public/browser/devtools_manager.h"
 
 namespace content {
 
 namespace {
 static int g_next_agent_host_id = 0;
+
+static std::string GenerateId() {

[pfeldman, 2013/03/04 11:43:01]

Why?

[Vladislav Kaznacheev, 2013/03/04 14:30:03]

Inlined back.
On 2013/03/04 11:43:01, pfeldman wrote:

+  return base::StringPrintf("%d", ++g_next_agent_host_id);
+}
+
 }  // namespace
 
 DevToolsAgentHostImpl::DevToolsAgentHostImpl()
     : close_listener_(NULL),
-      id_(++g_next_agent_host_id) {
+      id_(GenerateId()) {
 }
 
 void DevToolsAgentHostImpl::Attach() {
@@ -58,6 +65,22 @@ RenderViewHost* DevToolsAgentHostImpl::GetRenderViewHost() {
   return NULL;
 }
 
+std::string DevToolsAgentHostImpl::title() {
+  return "";
+}
+
+GURL DevToolsAgentHostImpl::url() {
+  return GURL();
+}
+
+GURL DevToolsAgentHostImpl::thumbnail_url() {
+  return GURL();
+}
+
+GURL DevToolsAgentHostImpl::favicon_url() {
+  return GURL();
+}
+
 void DevToolsAgentHostImpl::NotifyCloseListener() {
   if (close_listener_) {
     scoped_refptr<DevToolsAgentHostImpl> protect(this);