[DevTools] Implement Target.setDiscoverTargets method.

content/browser/devtools/forwarding_agent_host.cc

Index: content/browser/devtools/forwarding_agent_host.cc
diff --git a/content/browser/devtools/forwarding_agent_host.cc b/content/browser/devtools/forwarding_agent_host.cc
index cc07e72a0171efc075db42f139425cd154434bce..e46264fdf65a2ee52eb6d6247fc7160c3eb4db72 100644
--- a/content/browser/devtools/forwarding_agent_host.cc
+++ b/content/browser/devtools/forwarding_agent_host.cc
@@ -5,6 +5,7 @@
 #include "content/browser/devtools/forwarding_agent_host.h"
 
 #include "base/bind.h"
+#include "content/browser/devtools/devtools_manager.h"
 #include "content/browser/devtools/protocol/inspector_handler.h"
 
 namespace content {
@@ -14,9 +15,11 @@ ForwardingAgentHost::ForwardingAgentHost(
     std::unique_ptr<DevToolsExternalAgentProxyDelegate> delegate)
       : DevToolsAgentHostImpl(id),
         delegate_(std::move(delegate)) {
+  DevToolsManager::GetInstance()->AgentHostCreated(this);

[pfeldman, 2016/10/12 23:11:18]

Why not the DevToolsAgentHostImpl?

[dgozman, 2016/10/12 23:23:12]

Subclasses are not yet fully initialized in
DevToolsAgentHostImpl::DevToolsAgentHostImpl, and we can query GetType() or
something as a result of this call.

 }
 
 ForwardingAgentHost::~ForwardingAgentHost() {
+  DevToolsManager::GetInstance()->AgentHostDestroyed(this);
 }
 
 void ForwardingAgentHost::DispatchOnClientHost(const std::string& message) {