Virtual destructors should have virtual keyword.

chrome/browser/transport_security_persister.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.
 
 // TransportSecurityState maintains an in memory database containing the
 // list of hosts that currently have transport security enabled. This
 // singleton object deals with writing that data out to disk as needed and
 // loading it at startup.
 
 // At startup we need to load the transport security state from the
@@ skipping 35 matching lines @@
   explicit TransportSecurityPersister(bool readonly);
   void Initialize(net::TransportSecurityState* state,
                   const FilePath& profile_path);
 
   // Called by the TransportSecurityState when it changes its state.
   virtual void StateIsDirty(net::TransportSecurityState*);
 
  private:
   friend class base::RefCountedThreadSafe<TransportSecurityPersister>;
 
-  ~TransportSecurityPersister();
+  virtual ~TransportSecurityPersister();
 
   void Load();
   void CompleteLoad(const std::string& state);
 
   void Save();
   void CompleteSave(const std::string& state);
 
   // Used on the IO thread to coalesce writes to disk.
   ScopedRunnableMethodFactory<TransportSecurityPersister> save_coalescer_;
 
   scoped_refptr<net::TransportSecurityState>
       transport_security_state_;  // IO thread only.
 
   // The path to the file in which we store the serialised state.
   FilePath state_file_;
 
   // Whether or not we're in read-only mode.
   bool readonly_;
 };
 
 #endif  // CHROME_BROWSER_TRANSPORT_SECURITY_PERSISTER_H_