RefCounted types should not have public destructors, content/browser part 1

content/browser/resolve_proxy_msg_helper.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 CONTENT_BROWSER_RESOLVE_PROXY_MSG_HELPER_H_
 #define CONTENT_BROWSER_RESOLVE_PROXY_MSG_HELPER_H_
 #pragma once
 
 #include <deque>
 #include <string>
@@ skipping 19 matching lines @@
 // the stored IPC::Message pointers for pending requests.
 //
 // This object is expected to live on the IO thread.
 class CONTENT_EXPORT ResolveProxyMsgHelper
     : public content::BrowserMessageFilter {
  public:
   explicit ResolveProxyMsgHelper(net::URLRequestContextGetter* getter);
   // Constructor used by unittests.
   explicit ResolveProxyMsgHelper(net::ProxyService* proxy_service);
 
-  // Destruction cancels the current outstanding request, and clears the
-  // pending queue.
-  virtual ~ResolveProxyMsgHelper();
-
   // content::BrowserMessageFilter implementation
   virtual bool OnMessageReceived(const IPC::Message& message,
                                  bool* message_was_ok) OVERRIDE;
 
   void OnResolveProxy(const GURL& url, IPC::Message* reply_msg);
 
+ protected:
+  // Destruction cancels the current outstanding request, and clears the
+  // pending queue.
+  virtual ~ResolveProxyMsgHelper();
+
  private:
   // Callback for the ProxyService (bound to |callback_|).
   void OnResolveProxyCompleted(int result);
 
   // Starts the first pending request.
   void StartPendingRequest();
 
   // A PendingRequest is a resolve request that is in progress, or queued.
   struct PendingRequest {
    public:
@@ skipping 15 matching lines @@
 
   // FIFO queue of pending requests. The first entry is always the current one.
   typedef std::deque<PendingRequest> PendingRequestList;
   PendingRequestList pending_requests_;
 
   scoped_refptr<net::URLRequestContextGetter> context_getter_;
   net::ProxyService* proxy_service_;
 };
 
 #endif  // CONTENT_BROWSER_RESOLVE_PROXY_MSG_HELPER_H_