Virtual destructors should have virtual keyword.

chrome/browser/profiles/profile_io_data.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_BROWSER_PROFILES_PROFILE_IO_DATA_H_
 #define CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_
 #pragma once
 
 #include <set>
 #include "base/basictypes.h"
@@ skipping 76 matching lines @@
   // with a content::ResourceContext, and they want access to Chrome data for
   // that profile.
   HostContentSettingsMap* GetHostContentSettingsMap() const;
 
  protected:
   friend class base::RefCountedThreadSafe<ProfileIOData>;
 
   class RequestContext : public ChromeURLRequestContext {
    public:
     RequestContext();
-    ~RequestContext();
+    virtual ~RequestContext();
 
     // Setter is used to transfer ownership of the ProfileIOData to the context.
     void set_profile_io_data(const ProfileIOData* profile_io_data) {
       profile_io_data_ = profile_io_data;
     }
 
    private:
     scoped_refptr<const ProfileIOData> profile_io_data_;
   };
 
@@ skipping 148 matching lines @@
 
   // These are only valid in between LazyInitialize() and their accessor being
   // called.
   mutable scoped_refptr<RequestContext> main_request_context_;
   mutable scoped_refptr<RequestContext> extensions_request_context_;
 
   DISALLOW_COPY_AND_ASSIGN(ProfileIOData);
 };
 
 #endif  // CHROME_BROWSER_PROFILES_PROFILE_IO_DATA_H_