Extract debugger target enumeration into a separate class

content/browser/devtools/devtools_agent_host_impl.h

 // 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.
 
 #ifndef CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_AGENT_HOST_IMPL_H_
 #define CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_AGENT_HOST_IMPL_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
@@ skipping 23 matching lines @@
   void Detach();
   virtual void DispatchOnInspectorBackend(const std::string& message);
   void InspectElement(int x, int y);
   void AddMessageToConsole(ConsoleMessageLevel level,
                            const std::string& message);
 
   void set_close_listener(CloseListener* listener) {
     close_listener_ = listener;
   }
 
-  int id() { return id_; }
+  // DevToolsAgentHost implementation.
+  virtual std::string GetId() OVERRIDE { return id_; }

[jam, 2013/03/07 17:50:24]

i think you'll get a clang error for this, the implementation needs to move to
the cc file

 
-  // DevToolsAgentHost implementation.
   virtual RenderViewHost* GetRenderViewHost() OVERRIDE;
 
  protected:
   DevToolsAgentHostImpl();
   virtual ~DevToolsAgentHostImpl() {}
 
   virtual void SendMessageToAgent(IPC::Message* msg) = 0;
   virtual void NotifyClientAttaching() = 0;
   virtual void NotifyClientDetaching() = 0;
 
   void NotifyCloseListener();
 
   CloseListener* close_listener_;
 
  private:
-  int id_;
+  const std::string id_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_DEVTOOLS_DEVTOOLS_AGENT_HOST_IMPL_H_