Virtual destructors should have virtual keyword.

net/base/cert_verifier.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_CERT_VERIFIER_H_
 #define NET_BASE_CERT_VERIFIER_H_
 #pragma once
 
 #include <map>
 #include <string>
@@ skipping 53 matching lines @@
   };
 
   CertVerifier();
 
   // Used by unit tests to mock the current time.  Takes ownership of
   // |time_service|.
   explicit CertVerifier(TimeService* time_service);
 
   // When the verifier is destroyed, all certificate verifications requests are
   // canceled, and their completion callbacks will not be called.
-  ~CertVerifier();
+  virtual ~CertVerifier();
 
   // Verifies the given certificate against the given hostname.  Returns OK if
   // successful or an error code upon failure.
   //
   // The |*verify_result| structure, including the |verify_result->cert_status|
   // bitmask, is always filled out regardless of the return value.  If the
   // certificate has multiple errors, the corresponding status flags are set in
   // |verify_result->cert_status|, and the error code for the most serious
   // error is returned.
   //
@@ skipping 127 matching lines @@
 
   // Completion callback for when request to |cert_verifier_| completes.
   CompletionCallbackImpl<SingleRequestCertVerifier> callback_;
 
   DISALLOW_COPY_AND_ASSIGN(SingleRequestCertVerifier);
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_CERT_VERIFIER_H_