RefCounted types should not have public destructors

content/browser/geolocation/geolocation_provider_unittest.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 "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/string16.h"
@@ skipping 50 matching lines @@
 
   virtual void LoadAccessTokens(const LoadAccessTokensCallbackType& callback)
         OVERRIDE {
     callback.Run(AccessTokenSet(), NULL);
     event_->Signal();
   }
 
   virtual void SaveAccessToken(const GURL& server_url,
                                const string16& access_token) OVERRIDE {}
 
+ protected:
+  virtual ~TestingAccessTokenStore() {}
+
  private:
   base::WaitableEvent* event_;
 };
 
 class TestingDependencyFactory
     : public DefaultGeolocationArbitratorDependencyFactory {
  public:
   TestingDependencyFactory(base::WaitableEvent* event) : event_(event) {}
 
   virtual content::AccessTokenStore* NewAccessTokenStore() OVERRIDE {
     return new TestingAccessTokenStore(event_);
   }
 
   virtual LocationProviderBase* NewNetworkLocationProvider(
       content::AccessTokenStore* access_token_store,
       net::URLRequestContextGetter* context,
       const GURL& url,
       const string16& access_token) OVERRIDE {
     return new StartStopMockLocationProvider(event_);
   }
 
   virtual LocationProviderBase* NewSystemLocationProvider() OVERRIDE  {
     return NULL;
   }
 
+ protected:
+  virtual ~TestingDependencyFactory() {}
+
  private:
   base::WaitableEvent* event_;
 };
 
 class NullGeolocationObserver : public GeolocationObserver {
  public:
   // GeolocationObserver
   virtual void OnLocationUpdate(const content::Geoposition& position) {}
 };
 
@@ skipping 130 matching lines @@
   position.longitude = 34;
   position.accuracy = 56;
   position.timestamp = base::Time::Now();
   EXPECT_CALL(callback_wrapper, Callback(GeopositionEq(position)));
   provider_->OverrideLocationForTesting(position);
   // Wait for the providers to be stopped now that all clients are gone.
   WaitAndReset();
 }
 
 }  // namespace