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

Unified Diff: content/renderer/render_widget_fullscreen.cc

Issue 2450353004: Move ViewHostMsg_CreateWidget to mojom (Closed)
Patch Set: Addressed @tsepez's comment 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') | no next file » | 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 8a7b1e0127ae6341c70c5f7e16f25c5123c0a553..00d53724db95d739b2ad28cd6e5a725db3514247 100644
--- a/content/renderer/render_widget_fullscreen.cc
+++ b/content/renderer/render_widget_fullscreen.cc
@@ -5,6 +5,7 @@
#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;
@@ -40,12 +41,19 @@ WebWidget* RenderWidgetFullscreen::CreateWebWidget() {
return RenderWidget::CreateWebWidget(this);
}
+bool RenderWidgetFullscreen::SendIPC(int32_t opener_id, int32_t* routing_id) {
+ return RenderThread::Get()->Send(
+ new ViewHostMsg_CreateFullscreenWidget(opener_id, routing_id));
+}
+
bool RenderWidgetFullscreen::Init(int32_t opener_id) {
DCHECK(!GetWebWidget());
bool success = RenderWidget::DoInit(
opener_id, CreateWebWidget(),
- new ViewHostMsg_CreateFullscreenWidget(opener_id, &routing_id_));
+ base::Bind(&RenderWidgetFullscreen::SendIPC, base::Unretained(this),
+ opener_id, &routing_id_));
+
if (success) {
// TODO(fsamuel): This is a bit ugly. The |create_widget_message| should
// probably be factored out of RenderWidget::DoInit.
« no previous file with comments | « content/renderer/render_widget_fullscreen.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698