RenderWidget: hoist synchronous IPC out of Init methods.

content/renderer/render_widget_fullscreen.cc

 // 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"
 
@@ skipping 23 matching lines @@
                    false,
                    false) {}
 
 RenderWidgetFullscreen::~RenderWidgetFullscreen() {}
 
 WebWidget* RenderWidgetFullscreen::CreateWebWidget() {
   // TODO(boliu): Handle full screen render widgets here.
   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(),
-      base::Bind(&RenderWidgetFullscreen::SendIPC, base::Unretained(this),
-           opener_id, &routing_id_));
+  // Synchronous IPC to obtain a routing id for ourselves.
+  int32_t routing_id = MSG_ROUTING_NONE;
+  RenderThread::Get()->Send(
+        new ViewHostMsg_CreateFullscreenWidget(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.
-    SetRoutingID(routing_id_);
-    return true;
-  }
-  return false;
+  if (routing_id == MSG_ROUTING_NONE)
+    return false;
+
+  RenderWidget::Init(routing_id, opener_id, CreateWebWidget());
+  return true;
 }
 
 }  // namespace content