Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(916)

Unified Diff: content/public/renderer/render_view.h

Issue 10383262: RefCounted types should not have public destructors, delegate cleanup (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/renderer/render_view.h
diff --git a/content/public/renderer/render_view.h b/content/public/renderer/render_view.h
index 7d291a49ad9465c8fcf1f79b8511d9450ee82e82..3047c652826a6899be87106be0c0e3f9276e1bc8 100644
--- a/content/public/renderer/render_view.h
+++ b/content/public/renderer/render_view.h
@@ -48,8 +48,6 @@ class CONTENT_EXPORT RenderView : public IPC::Message::Sender {
// 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;
@@ -130,6 +128,9 @@ class CONTENT_EXPORT RenderView : public IPC::Message::Sender {
WebKit::WebFrame* frame,
const WebKit::WebURLRequest& request,
WebKit::WebNavigationPolicy policy) = 0;
+
+ protected:
+ virtual ~RenderView() {}
};
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698