Extract debugger target enumeration into a separate class

content/public/browser/devtools_target_list.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_PUBLIC_BROWSER_DEVTOOLS_TARGET_LIST_H_
+#define CONTENT_PUBLIC_BROWSER_DEVTOOLS_TARGET_LIST_H_
+
+#include <map>
+#include <string>
+
+#include "base/memory/ref_counted.h"
+#include "content/common/content_export.h"
+
+namespace content {
+
+class DevToolsAgentHost;
+
+class CONTENT_EXPORT DevToolsTargetList {
+ public:
+  static DevToolsTargetList* GetInstance();
+
+  typedef std::map<std::string, scoped_refptr<DevToolsAgentHost> > AgentsMap;
+
+  AgentsMap::iterator begin() { return agents_map_.begin(); }

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

I still prefer vector::iterator as the return type.

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

Discussed offline, agreed to keep as is.
On 2013/03/04 11:43:01, pfeldman wrote:

+
+  AgentsMap::iterator end()  { return agents_map_.end(); }
+
+  void Refresh();
+
+  void Register(DevToolsAgentHost* agent_host);
+
+  DevToolsAgentHost* Lookup(const std::string& id);
+
+ private:
+  AgentsMap agents_map_;
+
+  void GarbageCollect();
+};
+
+}  // namespace content
+
+#endif  // CONTENT_PUBLIC_BROWSER_DEVTOOLS_TARGET_LIST_H_