Refcounted types should not have public destructors, remoting/ edition

remoting/host/gaia_oauth_client.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 "remoting/host/gaia_oauth_client.h"
 
 #include "base/bind.h"
 #include "base/json/json_reader.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 13 matching lines @@
 
 namespace remoting {
 
 class GaiaOAuthClient::Core
     : public base::RefCountedThreadSafe<GaiaOAuthClient::Core> {
  public:
   Core(const std::string& gaia_url,
        net::URLRequestContextGetter* request_context_getter)
       : gaia_url_(gaia_url),
         request_context_getter_(request_context_getter),
-        delegate_(NULL) { }
-
-  virtual ~Core() { }
+        delegate_(NULL) {
+  }
 
   void RefreshToken(const OAuthClientInfo& oauth_client_info,
                     const std::string& refresh_token,
                     GaiaOAuthClient::Delegate* delegate);
 
  private:
+  friend class base::RefCountedThreadSafe<Core>;
+  virtual ~Core() {}
+
   void OnUrlFetchComplete(const net::URLRequestStatus& status,
                           int response_code,
                           const std::string& response);
 
   GURL gaia_url_;
   scoped_refptr<net::URLRequestContextGetter> request_context_getter_;
   GaiaOAuthClient::Delegate* delegate_;
   scoped_ptr<UrlFetcher> request_;
 };
 
@@ skipping 73 matching lines @@
 
 void GaiaOAuthClient::RefreshToken(const OAuthClientInfo& oauth_client_info,
                                    const std::string& refresh_token,
                                    Delegate* delegate) {
   return core_->RefreshToken(oauth_client_info,
                              refresh_token,
                              delegate);
 }
 
 }  // namespace remoting