Virtual destructors should have virtual keyword.

content/browser/cert_store.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_CERT_STORE_H_
 #define CONTENT_BROWSER_CERT_STORE_H_
 #pragma once
 
 #include <map>
 
@@ skipping 32 matching lines @@
 
   // NotificationObserver implementation.
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
                        const NotificationDetails& details);
 
  private:
   friend struct DefaultSingletonTraits<CertStore>;
 
   CertStore();
-  ~CertStore();
+  virtual ~CertStore();
 
   // Remove the specified cert from id_to_cert_ and cert_to_id_.
   // NOTE: the caller (RemoveCertsForRenderProcesHost) must hold cert_lock_.
   void RemoveCertInternal(int cert_id);
 
   // Removes all the certs associated with the specified process from the store.
   void RemoveCertsForRenderProcesHost(int render_process_host_id);
 
   typedef std::multimap<int, int> IDMap;
   typedef std::map<int, scoped_refptr<net::X509Certificate> > CertMap;
@@ skipping 11 matching lines @@
   int next_cert_id_;
 
   // This lock protects: process_to_ids_, id_to_processes_, id_to_cert_ and
   //                     cert_to_id_.
   base::Lock cert_lock_;
 
   DISALLOW_COPY_AND_ASSIGN(CertStore);
 };
 
 #endif  // CONTENT_BROWSER_CERT_STORE_H_