RefCounted types should not have public destructors, chromeos edition

chrome/browser/chromeos/mobile/mobile_activator.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_CHROMEOS_MOBILE_MOBILE_ACTIVATOR_H_
 #define CHROME_BROWSER_CHROMEOS_MOBILE_MOBILE_ACTIVATOR_H_
 #pragma once
 
 #include <map>
 #include <string>
 
 #include "base/file_path.h"
 #include "base/memory/singleton.h"
 #include "base/memory/weak_ptr.h"
 #include "base/observer_list.h"
 #include "chrome/browser/chromeos/cros/network_library.h"
 
 namespace chromeos {
 
 // Cellular plan config document.
 class CellularConfigDocument
     : public base::RefCountedThreadSafe<CellularConfigDocument> {
  public:
   CellularConfigDocument();
-  virtual ~CellularConfigDocument();
 
   // Return error message for a given code.
   std::string GetErrorMessage(const std::string& code);
   void LoadCellularConfigFile();
   const std::string& version() { return version_; }
 
  private:
+  friend class base::RefCountedThreadSafe<CellularConfigDocument>;
   typedef std::map<std::string, std::string> ErrorMap;
 
+  virtual ~CellularConfigDocument();
+
   void SetErrorMap(const ErrorMap& map);
   bool LoadFromFile(const FilePath& config_path);
 
   std::string version_;
   ErrorMap error_map_;
   base::Lock config_lock_;
 
   DISALLOW_COPY_AND_ASSIGN(CellularConfigDocument);
 };
 
@@ skipping 190 matching lines @@
   base::RepeatingTimer<MobileActivator> reconnect_timer_;
 
   ObserverList<Observer> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(MobileActivator);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_MOBILE_MOBILE_ACTIVATOR_H_