RefCounted types should not have public destructors, chromeos edition

chrome/browser/chromeos/login/owner_manager.h

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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_CHROMEOS_LOGIN_OWNER_MANAGER_H_
 #define CHROME_BROWSER_CHROMEOS_LOGIN_OWNER_MANAGER_H_
 #pragma once
 
 #include <string>
 #include <vector>
 
@@ skipping 32 matching lines @@
                                  const std::vector<uint8>& payload) = 0;
   };
 
   class KeyUpdateDelegate {
    public:
     // Called upon completion of a key update operation.
     virtual void OnKeyUpdated() = 0;
   };
 
   OwnerManager();
-  virtual ~OwnerManager();
 
   // Sets a new owner key from a provided memory buffer.
   void UpdateOwnerKey(const content::BrowserThread::ID thread_id,
                       const std::vector<uint8>& key,
                       KeyUpdateDelegate* d);
 
   // Pulls the owner's public key off disk and into memory.
   //
   // Call this on the FILE thread.
   void LoadOwnerKey();
@@ skipping 18 matching lines @@
   //
   // On success, calls d->OnKeyOpComplete() on |thread_id| with a
   // successful return code, passing an empty string for |payload|.
   // On failure, calls d->OnKeyOpComplete() on |thread_id| with an appropriate
   // error code, passing an empty string for |payload|.
   void Verify(const content::BrowserThread::ID thread_id,
               const std::string& data,
               const std::vector<uint8>& signature,
               Delegate* d);
 
+ protected:
+  virtual ~OwnerManager();
+
  private:
+  friend class base::RefCountedThreadSafe<OwnerManager>;
+
   // A helper method to send a notification on another thread.
   void SendNotification(int type,
                         const content::NotificationDetails& details);
 
   // Calls back a key update delegate on a given thread.
   void CallKeyUpdateDelegate(KeyUpdateDelegate* d) {
     d->OnKeyUpdated();
   }
 
   // A helper method to call back a delegte on another thread.
   void CallDelegate(Delegate* d,
                     const KeyOpCode return_code,
                     const std::vector<uint8>& payload) {
     d->OnKeyOpComplete(return_code, payload);
   }
 
   scoped_ptr<crypto::RSAPrivateKey> private_key_;
   std::vector<uint8> public_key_;
 
   scoped_refptr<OwnerKeyUtils> utils_;
 
   friend class OwnerManagerTest;
 
   DISALLOW_COPY_AND_ASSIGN(OwnerManager);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_LOGIN_OWNER_MANAGER_H_