Virtual destructors should have virtual keyword.

chrome/browser/net/url_request_mock_net_error_job.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.
 
 // A net::URLRequestJob class that simulates network errors (including https
 // related).
 // It is based on URLRequestMockHttpJob.
 
 #ifndef CHROME_BROWSER_NET_URL_REQUEST_MOCK_NET_ERROR_JOB_H_
 #define CHROME_BROWSER_NET_URL_REQUEST_MOCK_NET_ERROR_JOB_H_
@@ skipping 20 matching lines @@
   // contents and http-headers should be retrieved from.
   static void AddMockedURL(const GURL& url,
                            const std::wstring& base,
                            const std::vector<int>& errors,
                            net::X509Certificate* ssl_cert);
 
   // Removes the specified |url| from the list of mocked urls.
   static void RemoveMockedURL(const GURL& url);
 
  private:
-  ~URLRequestMockNetErrorJob();
+  virtual ~URLRequestMockNetErrorJob();
 
   static net::URLRequest::ProtocolFactory Factory;
 
   void StartAsync();
 
   // The errors to simulate.
   std::vector<int> errors_;
 
   // The certificate to use for SSL errors.
   scoped_refptr<net::X509Certificate> ssl_cert_;
 
   struct MockInfo;
   typedef std::map<GURL, MockInfo> URLMockInfoMap;
   static URLMockInfoMap url_mock_info_map_;
 
   ScopedRunnableMethodFactory<URLRequestMockNetErrorJob> method_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(URLRequestMockNetErrorJob);
 };
 
 #endif  // CHROME_BROWSER_NET_URL_REQUEST_MOCK_NET_ERROR_JOB_H_