DevTools: retain DTAH in all the targets to match their life time.

content/public/browser/devtools_agent_host.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_AGENT_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_DEVTOOLS_AGENT_HOST_H_
 
 #include <memory>
 #include <string>
 #include <vector>
@@ skipping 87 matching lines @@
   // protocol to discover other agent hosts.
   static scoped_refptr<DevToolsAgentHost> CreateForDiscovery();
 
   static bool IsDebuggerAttached(WebContents* web_contents);
 
   using List = std::vector<scoped_refptr<DevToolsAgentHost>>;
 
   // Returns all DevToolsAgentHosts content is aware of.
   static List GetOrCreateAll();
 
   using DiscoveryCallback = base::Callback<void(List)>;

[dgozman, 2017/04/22 00:00:45]

Remove?

 
   // Returns all possible DevToolsAgentHosts embedder is aware of.
-  static void DiscoverAllHosts(const DiscoveryCallback& callback);
+  static DevToolsAgentHost::List DiscoverAllHosts();
 
   // Starts remote debugging.
   // Takes ownership over |socket_factory|.
   // If |frontend_url| is empty, assumes it's bundled.
   // If |active_port_output_directory| is non-empty, it is assumed the
   // socket_factory was initialized with an ephemeral port (0). The
   // port selected by the OS will be written to a well-known file in
   // the output directory.
   static void StartRemoteDebuggingServer(
       std::unique_ptr<DevToolsSocketFactory> server_socket_factory,
@@ skipping 88 matching lines @@
   static void DetachAllClients();
 
  protected:
   friend class base::RefCounted<DevToolsAgentHost>;
   virtual ~DevToolsAgentHost() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_DEVTOOLS_AGENT_HOST_H_