Virtual destructors should have virtual keyword.

chrome/common/net/test_url_fetcher_factory.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 CHROME_COMMON_NET_TEST_URL_FETCHER_FACTORY_H_
 #define CHROME_COMMON_NET_TEST_URL_FETCHER_FACTORY_H_
 #pragma once
 
 #include <list>
 #include <map>
@@ skipping 31 matching lines @@
 // Note: if you don't know when your request objects will be created you
 // might want to use the FakeUrlFetcher and FakeUrlFetcherFactory classes
 // below.
 
 class TestURLFetcher : public URLFetcher {
  public:
   TestURLFetcher(int id,
                  const GURL& url,
                  RequestType request_type,
                  Delegate* d);
-  ~TestURLFetcher();
+  virtual ~TestURLFetcher();
 
   // Overriden to do nothing. It is assumed the caller will notify the delegate.
   virtual void Start() {}
 
   // Overriden to cache the chunks uploaded. Caller can read back the uploaded
   // chunks with the upload_data() accessor.
   virtual void AppendChunkToUpload(const std::string& data, bool is_last_chunk);
 
   // Unique ID in our factory.
   int id() const { return id_; }
@@ skipping 132 matching lines @@
   void ClearFakeReponses();
 
  private:
   typedef std::map<GURL, std::pair<std::string, bool> > FakeResponseMap;
   FakeResponseMap fake_responses_;
 
   DISALLOW_COPY_AND_ASSIGN(FakeURLFetcherFactory);
 };
 
 #endif  // CHROME_COMMON_NET_TEST_URL_FETCHER_FACTORY_H_