No need to have DevToolsManager reference counted

content/browser/debugger/devtools_manager.h

 // Copyright (c) 2011 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_BROWSER_DEBUGGER_DEVTOOLS_MANAGER_H_
 #define CONTENT_BROWSER_DEBUGGER_DEVTOOLS_MANAGER_H_
 #pragma once
 
 #include <map>
 #include <string>
 
-#include "base/memory/ref_counted.h"
 #include "content/browser/debugger/devtools_client_host.h"
 #include "content/common/notification_observer.h"
 #include "content/common/notification_registrar.h"
 
 namespace IPC {
 class Message;
 }
 
 class DevToolsNetLogObserver;
 class GURL;
 class IOThread;
 class PrefService;
 class RenderViewHost;
 class TabContents;
 
 typedef std::map<std::string, std::string> DevToolsRuntimeProperties;
 
 // This class is a singleton that manages DevToolsClientHost instances and
 // routes messages between developer tools clients and agents.
 class DevToolsManager : public DevToolsClientHost::CloseListener,
-                        public NotificationObserver,
-                        public base::RefCounted<DevToolsManager> {
+                        public NotificationObserver {
  public:
   static DevToolsManager* GetInstance();
 
   DevToolsManager();
+  virtual ~DevToolsManager();
 
   // Returns DevToolsClientHost registered for |inspected_rvh| or NULL if
   // there is no alive DevToolsClientHost registered for |inspected_rvh|.
   DevToolsClientHost* GetDevToolsClientHostFor(RenderViewHost* inspected_rvh);
 
   // Registers new DevToolsClientHost for |inspected_rvh|. There must be no
   // other DevToolsClientHosts registered for the RenderViewHost at the moment.
   void RegisterDevToolsClientHostFor(RenderViewHost* inspected_rvh,
                                      DevToolsClientHost* client_host);
   void UnregisterDevToolsClientHostFor(RenderViewHost* inspected_rvh);
@@ skipping 25 matching lines @@
   int DetachClientHost(RenderViewHost* from_rvh);
 
   // Attaches orphan client host to new render view host.
   void AttachClientHost(int client_host_cookie,
                         RenderViewHost* to_rvh);
 
   // Closes all open developer tools windows.
   void CloseAllClientHosts();
 
  private:
-  friend class base::RefCounted<DevToolsManager>;
-
-  virtual ~DevToolsManager();
-
   // DevToolsClientHost::CloseListener override.
   // This method will remove all references from the manager to the
   // DevToolsClientHost and unregister all listeners related to the
   // DevToolsClientHost.
   virtual void ClientHostClosing(DevToolsClientHost* host);
 
   // Overridden from NotificationObserver:
   virtual void Observe(int type,
                        const NotificationSource& source,
                        const NotificationDetails& details);
@@ skipping 35 matching lines @@
       OrphanClientHosts;
   OrphanClientHosts orphan_client_hosts_;
   int last_orphan_cookie_;
 
   NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(DevToolsManager);
 };
 
 #endif  // CONTENT_BROWSER_DEBUGGER_DEVTOOLS_MANAGER_H_