Virtual destructors should have virtual keyword.

content/browser/geolocation/location_arbitrator.h

 // Copyright (c) 2011 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 CONTENT_BROWSER_GEOLOCATION_LOCATION_ARBITRATOR_H_
 #define CONTENT_BROWSER_GEOLOCATION_LOCATION_ARBITRATOR_H_
 #pragma once
 
 #include "base/memory/scoped_vector.h"
 #include "base/string16.h"
@@ skipping 22 matching lines @@
 class GeolocationArbitrator : public LocationProviderBase::ListenerInterface {
  public:
   // Number of milliseconds newer a location provider has to be that it's worth
   // switching to this location provider on the basis of it being fresher
   // (regardles of relative accuracy). Public for tests.
   static const int64 kFixStaleTimeoutMilliseconds;
 
   // Defines a function that returns the current time.
   typedef base::Time (*GetTimeNow)();
 
-  ~GeolocationArbitrator();
+  virtual ~GeolocationArbitrator();
 
   static GeolocationArbitrator* Create(GeolocationObserver* observer);
 
   // See more details in geolocation_provider.
   void StartProviders(const GeolocationObserverOptions& options);
   void StopProviders();
 
   // Called everytime permission is granted to a page for using geolocation.
   // This may either be through explicit user action (e.g. responding to the
   // infobar prompt) or inferred from a persisted site permission.
@@ skipping 41 matching lines @@
   const LocationProviderBase* position_provider_;
   GURL most_recent_authorized_frame_;
   CancelableRequestConsumer request_consumer_;
   // The current best estimate of our position.
   Geoposition position_;
 
   DISALLOW_COPY_AND_ASSIGN(GeolocationArbitrator);
 };
 
 #endif  // CONTENT_BROWSER_GEOLOCATION_LOCATION_ARBITRATOR_H_