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(

[pfeldman, 2013/03/29 14:59:23]

You no longer need this.

[Vladislav Kaznacheev, 2013/03/29 15:15:35]

Done.

         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 527 matching lines @@
     devtools_http_handler_->Stop();
     return result;
   }
 };
 
 IN_PROC_BROWSER_TEST_F(DISABLED_RemoteDebuggingTest, TargetList) {
   ASSERT_TRUE(RunExtensionTest("target_list"));
 }
 
 }  // namespace