Virtual destructors should have virtual keyword.

net/base/ssl_client_auth_cache.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 NET_BASE_SSL_CLIENT_AUTH_CACHE_H_
 #define NET_BASE_SSL_CLIENT_AUTH_CACHE_H_
 #pragma once
 
 #include <string>
 #include <map>
 
 #include "base/memory/ref_counted.h"
 #include "net/base/cert_database.h"
 #include "net/base/net_api.h"
 
 namespace net {
 
 class X509Certificate;
 
 // The SSLClientAuthCache class is a simple cache structure to store SSL
 // client certificates. Provides lookup, insertion, and deletion of entries.
 // The parameter for doing lookups, insertions, and deletions is the server's
 // host and port.
 //
 // TODO(wtc): This class is based on FtpAuthCache.  We can extract the common
 // code to a template class.
 class NET_TEST SSLClientAuthCache : public CertDatabase::Observer {
  public:
   SSLClientAuthCache();
-  ~SSLClientAuthCache();
+  virtual ~SSLClientAuthCache();
 
   // Checks for a client certificate preference for SSL server at |server|.
   // Returns true if a preference is found, and sets |*certificate| to the
   // desired client certificate. The desired certificate may be NULL, which
   // indicates a preference to not send any certificate to |server|.
   // If a certificate preference is not found, returns false.
   bool Lookup(const std::string& server,
               scoped_refptr<X509Certificate>* certificate);
 
   // Add a client certificate for |server| to the cache. If there is already
@@ skipping 13 matching lines @@
   typedef scoped_refptr<X509Certificate> AuthCacheValue;
   typedef std::map<AuthCacheKey, AuthCacheValue> AuthCacheMap;
 
   // internal representation of cache, an STL map.
   AuthCacheMap cache_;
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_SSL_CLIENT_AUTH_CACHE_H_