RefCounted types should not have public destructors, chrome/browser/extensions

chrome/browser/extensions/api/api_resource_event_notifier.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 CHROME_BROWSER_EXTENSIONS_API_API_RESOURCE_EVENT_NOTIFIER_H_
 #define CHROME_BROWSER_EXTENSIONS_API_API_RESOURCE_EVENT_NOTIFIER_H_
 #pragma once
 
 #include <string>
 
@@ skipping 30 matching lines @@
 
 // APIResourceEventNotifier knows how to send an event to a specific app's
 // onEvent handler. It handles all platform-API events.
 class APIResourceEventNotifier
     : public base::RefCountedThreadSafe<APIResourceEventNotifier> {
  public:
   APIResourceEventNotifier(ExtensionEventRouter* router,
                            Profile* profile,
                            const std::string& src_extension_id, int src_id,
                            const GURL& src_url);
-  virtual ~APIResourceEventNotifier();
 
   virtual void OnConnectComplete(int result_code);
 
   // Takes ownership of data.
   virtual void OnDataRead(int result_code, base::ListValue* data);
 
   virtual void OnWriteComplete(int result_code);
 
   static std::string APIResourceEventTypeToString(
       APIResourceEventType event_type);
 
  private:
+  friend class base::RefCountedThreadSafe<APIResourceEventNotifier>;
+  friend class MockAPIResourceEventNotifier;
+
+  virtual ~APIResourceEventNotifier();
+
   void DispatchEvent(DictionaryValue* event);
   void DispatchEventOnUIThread(DictionaryValue* event);
   DictionaryValue* CreateAPIResourceEvent(APIResourceEventType event_type);
 
   void SendEventWithResultCode(APIResourceEventType event_type,
                                int result_code);
 
   ExtensionEventRouter* router_;
   Profile* profile_;
   std::string src_extension_id_;
   int src_id_;
   GURL src_url_;
 
   DISALLOW_COPY_AND_ASSIGN(APIResourceEventNotifier);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_API_RESOURCE_EVENT_NOTIFIER_H_