Virtual destructors should have virtual keyword.

chrome/browser/sync/glue/http_bridge.h

Index: chrome/browser/sync/glue/http_bridge.h
diff --git a/chrome/browser/sync/glue/http_bridge.h b/chrome/browser/sync/glue/http_bridge.h
index 4c013925560373ea6d15fedc722be384c2c85e34..a4ee3eb5082b34ae127641b4967f561b89c6851d 100644
--- a/chrome/browser/sync/glue/http_bridge.h
+++ b/chrome/browser/sync/glue/http_bridge.h
@@ -61,7 +61,7 @@ class HttpBridge : public base::RefCountedThreadSafe<HttpBridge>,
 
    private:
     // The destructor MUST be called on the IO thread.
-    ~RequestContext();
+    virtual ~RequestContext();
 
     std::string user_agent_;
     net::URLRequestContext* baseline_context_;
@@ -83,7 +83,7 @@ class HttpBridge : public base::RefCountedThreadSafe<HttpBridge>,
     virtual scoped_refptr<base::MessageLoopProxy> GetIOMessageLoopProxy() const;
 
    private:
-    ~RequestContextGetter() {}
+    virtual ~RequestContextGetter() {}
 
     // User agent to apply to the net::URLRequestContext.
     std::string user_agent_;