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

Unified Diff: content/renderer/render_widget_fullscreen.cc

Issue 2498463002: RenderWidget/RenderView: encapsulate ViewHostMsg_Show, etc, in a callback (Closed)
Patch Set: Remove RenderWidgetFullscreen Created 4 years, 1 month 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/render_widget_fullscreen_pepper.h » ('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
deleted file mode 100644
index 436744debb1b22d5a39c7368c7117f7c111a4c29..0000000000000000000000000000000000000000
--- a/content/renderer/render_widget_fullscreen.cc
+++ /dev/null
@@ -1,53 +0,0 @@
-// 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.
-
-#include "content/renderer/render_widget_fullscreen.h"
-
-#include "content/common/view_messages.h"
-#include "content/renderer/render_thread_impl.h"
-#include "third_party/WebKit/public/web/WebWidget.h"
-
-using blink::WebWidget;
-
-namespace content {
-
-void RenderWidgetFullscreen::show(blink::WebNavigationPolicy) {
- DCHECK(!did_show_) << "received extraneous Show call";
- DCHECK_NE(MSG_ROUTING_NONE, routing_id());
- DCHECK_NE(MSG_ROUTING_NONE, opener_id_);
-
- if (!did_show_) {
- did_show_ = true;
- Send(new ViewHostMsg_ShowFullscreenWidget(opener_id_, routing_id()));
- SetPendingWindowRect(initial_rect_);
- }
-}
-
-RenderWidgetFullscreen::RenderWidgetFullscreen(
- int32_t widget_routing_id,
- CompositorDependencies* compositor_deps,
- const ScreenInfo& screen_info)
- : RenderWidget(widget_routing_id,
- compositor_deps,
- blink::WebPopupTypeNone,
- screen_info,
- false,
- false,
- false) {}
-
-RenderWidgetFullscreen::~RenderWidgetFullscreen() {}
-
-WebWidget* RenderWidgetFullscreen::CreateWebWidget() {
- // TODO(boliu): Handle full screen render widgets here.
- return RenderWidget::CreateWebWidget(this);
-}
-
-bool RenderWidgetFullscreen::Init(int32_t opener_id) {
- DCHECK(!GetWebWidget());
-
- RenderWidget::Init(opener_id, CreateWebWidget());
- return true;
-}
-
-} // namespace content
« no previous file with comments | « content/renderer/render_widget_fullscreen.h ('k') | content/renderer/render_widget_fullscreen_pepper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698