Remove redundant DevToolsManager methods and clean up its clients.

chrome/browser/devtools/devtools_sanity_browsertest.cc

 // 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.
 
 #include "base/bind.h"
 #include "base/cancelable_callback.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/path_service.h"
@@ skipping 134 matching lines @@
     window_ = DevToolsWindow::OpenDevToolsWindow(inspected_rvh_);
     observer.Wait();
   }
 
   WebContents* GetInspectedTab() {
     return browser()->tab_strip_model()->GetWebContentsAt(0);
   }
 
   void CloseDevToolsWindow() {
     DevToolsManager* devtools_manager = DevToolsManager::GetInstance();
-    // UnregisterDevToolsClientHostFor may destroy window_ so store the browser
-    // first.
+    // CloseAllClientHosts may destroy window_ so store the browser first.
     Browser* browser = window_->browser();
-    scoped_refptr<DevToolsAgentHost> agent(
-        DevToolsAgentHost::GetOrCreateFor(inspected_rvh_));
-    devtools_manager->UnregisterDevToolsClientHostFor(agent);
-
+    devtools_manager->CloseAllClientHosts();
     // Wait only when DevToolsWindow has a browser. For docked DevTools, this
     // is NULL and we skip the wait.
     if (browser)
       BrowserClosedObserver close_observer(browser);
   }
 
   DevToolsWindow* window_;
   RenderViewHost* inspected_rvh_;
 };
 
@@ skipping 526 matching lines @@
     devtools_http_handler_->Stop();
     return result;
   }
 };
 
 IN_PROC_BROWSER_TEST_F(DISABLED_RemoteDebuggingTest, TargetList) {
   ASSERT_TRUE(RunExtensionTest("target_list"));
 }
 
 }  // namespace