RefCounted types should not have public destructors, delegate cleanup

content/public/renderer/render_view.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_RENDERER_RENDER_VIEW_H_
 #define CONTENT_PUBLIC_RENDERER_RENDER_VIEW_H_
 
 #include "base/basictypes.h"
 #include "content/common/content_export.h"
 #include "ipc/ipc_message.h"
@@ skipping 30 matching lines @@
 
 class CONTENT_EXPORT RenderView : public IPC::Message::Sender {
  public:
   // Returns the RenderView containing the given WebView.
   static RenderView* FromWebView(WebKit::WebView* webview);
 
   // Visit all RenderViews with a live WebView (i.e., RenderViews that have
   // been closed but not yet destroyed are excluded).
   static void ForEach(RenderViewVisitor* visitor);
 
-  virtual ~RenderView() {}
-
   // Get the routing ID of the view.
   virtual int GetRoutingID() const = 0;
 
   // Page IDs allow the browser to identify pages in each renderer process for
   // keeping back/forward history in sync.
   // Note that this is NOT updated for every main frame navigation, only for
   // "regular" navigations that go into session history. In particular, client
   // redirects, like the page cycler uses (document.location.href="foo") do not
   // count as regular navigations and do not increment the page id.
   virtual int GetPageId() const = 0;
@@ skipping 60 matching lines @@
   // Displays a modal alert dialog containing the given message.  Returns
   // once the user dismisses the dialog.
   virtual void RunModalAlertDialog(WebKit::WebFrame* frame,
                                    const WebKit::WebString& message) = 0;
 
   // The client should handle the navigation externally.
   virtual void LoadURLExternally(
       WebKit::WebFrame* frame,
       const WebKit::WebURLRequest& request,
       WebKit::WebNavigationPolicy policy) = 0;
+
+ protected:
+  virtual ~RenderView() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_RENDERER_RENDER_VIEW_H_