Virtual destructors should have virtual keyword.

content/browser/geolocation/geolocation_provider.h

 // Copyright (c) 2010 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_GEOLOCATION_PROVIDER_H_
 #define CONTENT_BROWSER_GEOLOCATION_GEOLOCATION_PROVIDER_H_
 #pragma once
 
 #include <map>
 
@@ skipping 39 matching lines @@
   // Gets a pointer to the singleton instance of the location relayer, which
   // is in turn bound to the browser's global context objects. Ownership is NOT
   // returned.
   static GeolocationProvider* GetInstance();
 
   typedef std::map<GeolocationObserver*, GeolocationObserverOptions>
       ObserverMap;
 
  private:
   friend struct DefaultSingletonTraits<GeolocationProvider>;
-  ~GeolocationProvider();
+  virtual ~GeolocationProvider();
 
   bool OnClientThread() const;
   bool OnGeolocationThread() const;
 
   // When the observer list changes, we may start the thread and the required
   // providers, or stop them.
   void OnObserversChanged();
 
   // Stop the providers when there are no more observers. Note that once our
   // thread is started, we'll keep it alive (but with no pending messages).
@@ skipping 20 matching lines @@
   GURL most_recent_authorized_frame_;
   Geoposition position_;
 
   // Only to be used on the geolocation thread.
   GeolocationArbitrator* arbitrator_;
 
   DISALLOW_COPY_AND_ASSIGN(GeolocationProvider);
 };
 
 #endif  // CONTENT_BROWSER_GEOLOCATION_GEOLOCATION_PROVIDER_H_