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

Unified Diff: content/renderer/render_widget_fullscreen.cc

Issue 10071038: RefCounted types should not have public destructors, content/browser part 2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Copyright bump Created 8 years, 8 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
« no previous file with comments | « content/renderer/render_widget_fullscreen.h ('k') | content/renderer/renderer_main.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_widget_fullscreen.cc
diff --git a/content/renderer/render_widget_fullscreen.cc b/content/renderer/render_widget_fullscreen.cc
index 7916889282ccb3cae6239f9bfe14051ede4d3f68..7030e5ded7457c6748f9b584aab8b39e4f3fee4a 100644
--- a/content/renderer/render_widget_fullscreen.cc
+++ b/content/renderer/render_widget_fullscreen.cc
@@ -18,21 +18,6 @@ RenderWidgetFullscreen* RenderWidgetFullscreen::Create(int32 opener_id) {
return widget.release();
}
-WebWidget* RenderWidgetFullscreen::CreateWebWidget() {
- // TODO(boliu): Handle full screen render widgets here.
- return RenderWidget::CreateWebWidget(this);
-}
-
-void RenderWidgetFullscreen::Init(int32 opener_id) {
- DCHECK(!webwidget_);
-
- RenderWidget::DoInit(
- opener_id,
- CreateWebWidget(),
- new ViewHostMsg_CreateFullscreenWidget(
- opener_id, &routing_id_, &surface_id_));
-}
-
void RenderWidgetFullscreen::show(WebKit::WebNavigationPolicy) {
DCHECK(!did_show_) << "received extraneous Show call";
DCHECK_NE(MSG_ROUTING_NONE, routing_id_);
@@ -48,3 +33,20 @@ void RenderWidgetFullscreen::show(WebKit::WebNavigationPolicy) {
RenderWidgetFullscreen::RenderWidgetFullscreen()
: RenderWidget(WebKit::WebPopupTypeNone, WebKit::WebScreenInfo()) {
}
+
+RenderWidgetFullscreen::~RenderWidgetFullscreen() {}
+
+WebWidget* RenderWidgetFullscreen::CreateWebWidget() {
+ // TODO(boliu): Handle full screen render widgets here.
+ return RenderWidget::CreateWebWidget(this);
+}
+
+void RenderWidgetFullscreen::Init(int32 opener_id) {
+ DCHECK(!webwidget_);
+
+ RenderWidget::DoInit(
+ opener_id,
+ CreateWebWidget(),
+ new ViewHostMsg_CreateFullscreenWidget(
+ opener_id, &routing_id_, &surface_id_));
+}
« no previous file with comments | « content/renderer/render_widget_fullscreen.h ('k') | content/renderer/renderer_main.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698