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::CreateFullscreenWidget(int32_t opener_id,
-                                                    int32_t* routing_id) {
-  RenderThreadImpl::current_render_message_filter()->CreateFullscreenWidget(
-      opener_id, routing_id);
-  return true;
-}
-
 bool RenderWidgetFullscreen::Init(int32_t opener_id) {
   DCHECK(!GetWebWidget());
 
-  bool success = RenderWidget::DoInit(
-      opener_id, CreateWebWidget(),
-      base::Bind(&RenderWidgetFullscreen::CreateFullscreenWidget,
-          base::Unretained(this), opener_id, &routing_id_));
+  // Synchronous IPC to obtain a routing id for ourselves.
+  int32_t routing_id = MSG_ROUTING_NONE;
+  if (!RenderThreadImpl::current_render_message_filter()
+           ->CreateFullscreenWidget(opener_id, &routing_id)) {
+    return false;
+  }
 
-  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;
+  RenderWidget::InitRoutingID(routing_id);
+  RenderWidget::Init(opener_id, CreateWebWidget());
+  return true;
 }
 
 }  // namespace content