RefCounted types should not have public destructors, content/browser part 1

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/memory/singleton.h"
 #include "base/synchronization/waitable_event.h"
 #include "content/browser/geolocation/arbitrator_dependency_factories_for_test.h"
 #include "content/browser/geolocation/fake_access_token_store.h"
 #include "content/browser/geolocation/geolocation_provider.h"
 #include "content/browser/geolocation/location_arbitrator.h"
@@ skipping 120 matching lines @@
       const GURL& url,
       const string16& access_token) {
     return new StartStopMockLocationProvider(test_loop_);
   }
 
   virtual LocationProviderBase* NewSystemLocationProvider() {
     return NULL;
   }
 
  private:
+  virtual ~MockDependencyFactory() {}
+
   MessageLoop* test_loop_;
   scoped_refptr<AccessTokenStore> access_token_store_;
 };
 
 TEST_F(GeolocationProviderTest, StartStop) {
   scoped_refptr<FakeAccessTokenStore> fake_access_token_store =
       new FakeAccessTokenStore;
   scoped_refptr<GeolocationArbitratorDependencyFactory> dependency_factory =
       new MockDependencyFactory(&message_loop_, fake_access_token_store.get());
   base::WaitableEvent event(false, false);
@@ skipping 53 matching lines @@
       testing::Field(
           &Geoposition::error_code,
           testing::Eq(Geoposition::ERROR_CODE_POSITION_UNAVAILABLE))));
   provider_->AddObserver(&mock_observer, GeolocationObserverOptions());
   provider_->RemoveObserver(&mock_observer);
 
   GeolocationArbitrator::SetDependencyFactoryForTest(NULL);
 }
 
 }  // namespace