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

Unified Diff: content/renderer/render_view_impl.cc

Issue 2496633004: Revert of RenderWidget: hoist synchronous IPC out of Init methods. (Closed)
Patch Set: 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 | « no previous file | content/renderer/render_widget.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index ebdb7894f52162ccf6f769396589b1997cedeaf0..14a3e97ea2d7698f053d971bfdde86b86d814da8 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -697,20 +697,21 @@
void RenderViewImpl::Initialize(const mojom::CreateViewParams& params,
bool was_created_by_renderer) {
- RenderWidget::InitRoutingID(params.view_id);
-
- int opener_view_routing_id = MSG_ROUTING_NONE;
+ SetRoutingID(params.view_id);
+
+ int opener_view_routing_id;
WebFrame* opener_frame = RenderFrameImpl::ResolveOpener(
params.opener_frame_route_id, &opener_view_routing_id);
- if (!was_created_by_renderer)
- opener_view_routing_id = MSG_ROUTING_NONE;
+ if (opener_view_routing_id != MSG_ROUTING_NONE && was_created_by_renderer)
+ opener_id_ = opener_view_routing_id;
display_mode_ = params.initial_size.display_mode;
webview_ =
WebView::create(this, is_hidden() ? blink::WebPageVisibilityStateHidden
: blink::WebPageVisibilityStateVisible);
- RenderWidget::Init(opener_view_routing_id, webview_->widget());
+ RenderWidget::DoInit(MSG_ROUTING_NONE, webview_->widget(),
+ CreateWidgetCallback());
g_view_map.Get().insert(std::make_pair(webview(), this));
g_routing_id_view_map.Get().insert(std::make_pair(GetRoutingID(), this));
@@ -2174,8 +2175,6 @@
}
int RenderViewImpl::GetRoutingID() const {
- DCHECK_NE(routing_id(), MSG_ROUTING_NONE)
- << "Tried to obtain routing ID before it was set.";
return routing_id();
}
« no previous file with comments | « no previous file | content/renderer/render_widget.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698