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

BUG=123295
TEST=none


Review URL: http://codereview.chromium.org/10068037

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134395 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 5 deletions

diff --git a/content/browser/resolve_proxy_msg_helper.h b/content/browser/resolve_proxy_msg_helper.h
index 920ff74..8b4fcb7 100644
--- a/content/browser/resolve_proxy_msg_helper.h
+++ b/content/browser/resolve_proxy_msg_helper.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 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.
 
@@ -37,16 +37,17 @@ class CONTENT_EXPORT ResolveProxyMsgHelper
   // 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);