Give the main frame a RenderWidget.

content/common/view_messages.h

 // 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.
 
 // IPC messages for page rendering.
 // Multiply-included message file, hence no include guard.
 
 #include "base/memory/shared_memory.h"
 #include "base/process/process.h"
 #include "base/strings/string16.h"
@@ skipping 343 matching lines @@
 
   // The window features to use for the new view.
   IPC_STRUCT_MEMBER(blink::WebWindowFeatures, features)
 
   // The additional window features to use for the new view. We pass these
   // separately from |features| above because we cannot serialize WebStrings
   // over IPC.
   IPC_STRUCT_MEMBER(std::vector<base::string16>, additional_features)
 IPC_STRUCT_END()
 
+IPC_STRUCT_BEGIN(ViewHostMsg_CreateWindow_Reply)
+  // The ID of the view to be created. If the ID is MSG_ROUTING_NONE, then the
+  // view couldn't be created.
+  IPC_STRUCT_MEMBER(int32, route_id, MSG_ROUTING_NONE)
+
+  // The ID of the view's rendering surface.
+  // TODO(dcheng): Remove this, once RenderView is no longer a RenderWidget.
+  IPC_STRUCT_MEMBER(int32, surface_id)
+
+  // The ID of the main frame hosted in the view.
+  IPC_STRUCT_MEMBER(int32, main_frame_route_id, MSG_ROUTING_NONE)
+
+  // The ID of the widget for the main frame.
+  IPC_STRUCT_MEMBER(int32, main_frame_widget_route_id, MSG_ROUTING_NONE)
+
+  // The ID of the main frame widget's rendering surface.
+  IPC_STRUCT_MEMBER(int32, widget_surface_id)
+
+  // TODO(dcheng): No clue. This is kind of duplicated from ViewMsg_New_Params.
+  IPC_STRUCT_MEMBER(int64, cloned_session_storage_namespace_id)
+IPC_STRUCT_END()
+
 IPC_STRUCT_BEGIN(ViewHostMsg_CreateWorker_Params)
   // URL for the worker script.
   IPC_STRUCT_MEMBER(GURL, url)
 
   // Name for a SharedWorker, otherwise empty string.
   IPC_STRUCT_MEMBER(base::string16, name)
 
   // Security policy used in the worker.
   IPC_STRUCT_MEMBER(base::string16, content_security_policy)
 
@@ skipping 119 matching lines @@
 IPC_STRUCT_END()
 
 IPC_STRUCT_BEGIN(ViewMsg_New_Params)
   // Renderer-wide preferences.
   IPC_STRUCT_MEMBER(content::RendererPreferences, renderer_preferences)
 
   // Preferences for this view.
   IPC_STRUCT_MEMBER(content::WebPreferences, web_preferences)
 
   // The ID of the view to be created.
-  IPC_STRUCT_MEMBER(int32, view_id)
+  IPC_STRUCT_MEMBER(int32, view_id, MSG_ROUTING_NONE)
+
+  // The ID of the view's rendering surface.
+  // TODO(dcheng): Remove this, once RenderView is no longer a RenderWidget.
+  IPC_STRUCT_MEMBER(int32, view_surface_id)
 
   // The ID of the main frame hosted in the view.
-  IPC_STRUCT_MEMBER(int32, main_frame_routing_id)
+  IPC_STRUCT_MEMBER(int32, main_frame_routing_id, MSG_ROUTING_NONE)
 
-  // The ID of the rendering surface.
-  IPC_STRUCT_MEMBER(int32, surface_id)
+  // The ID of the widget for the main frame.
+  IPC_STRUCT_MEMBER(int32, main_frame_widget_routing_id, MSG_ROUTING_NONE)
+
+  // The ID of the main frame widget's rendering surface.
+  IPC_STRUCT_MEMBER(int32, widget_surface_id)
 
   // The session storage namespace ID this view should use.
   IPC_STRUCT_MEMBER(int64, session_storage_namespace_id)
 
   // The route ID of the opener RenderFrame or RenderFrameProxy, if we need to
   // set one (MSG_ROUTING_NONE otherwise).
-  IPC_STRUCT_MEMBER(int, opener_frame_route_id)
+  IPC_STRUCT_MEMBER(int, opener_frame_route_id, MSG_ROUTING_NONE)
 
   // Whether the RenderView should initially be swapped out.
   IPC_STRUCT_MEMBER(bool, swapped_out)
 
   // Carries replicated information, such as frame name and sandbox flags, for
   // this view's main frame, which will be a proxy in |swapped_out|
   // views when in --site-per-process mode, or a RenderFrame in all other
   // cases.
   IPC_STRUCT_MEMBER(content::FrameReplicationState, replicated_frame_state)
 
   // The ID of the proxy object for the main frame in this view. It is only
   // used if |swapped_out| is true.
-  IPC_STRUCT_MEMBER(int32, proxy_routing_id)
+  IPC_STRUCT_MEMBER(int32, proxy_routing_id, MSG_ROUTING_NONE)
 
   // Whether the RenderView should initially be hidden.
   IPC_STRUCT_MEMBER(bool, hidden)
 
   // Whether the RenderView will never be visible.
   IPC_STRUCT_MEMBER(bool, never_visible)
 
   // Whether the window associated with this view was created with an opener.
   IPC_STRUCT_MEMBER(bool, window_was_created_with_opener)
 
@@ skipping 454 matching lines @@
 // Messages sent from the renderer to the browser.
 
 // Sent by renderer to request a ViewMsg_BeginFrame message for upcoming
 // display events. If |enabled| is true, the BeginFrame message will continue
 // to be be delivered until the notification is disabled.
 IPC_MESSAGE_ROUTED1(ViewHostMsg_SetNeedsBeginFrames,
                     bool /* enabled */)
 
 // Sent by the renderer when it is creating a new window.  The browser creates
 // a tab for it and responds with a ViewMsg_CreatingNew_ACK.  If route_id is
 // MSG_ROUTING_NONE, the view couldn't be created.

[ncarter (slow), 2015/09/03 20:24:37]

route_id -> |reply.route_id| ?

[dcheng, 2015/09/03 21:29:24]

Doh, I thought I changed this one. Done.

-IPC_SYNC_MESSAGE_CONTROL1_4(ViewHostMsg_CreateWindow,
+IPC_SYNC_MESSAGE_CONTROL1_1(ViewHostMsg_CreateWindow,
                             ViewHostMsg_CreateWindow_Params,
-                            int /* route_id */,
-                            int /* main_frame_route_id */,
-                            int32 /* surface_id */,
-                            int64 /* cloned_session_storage_namespace_id */)
+                            ViewHostMsg_CreateWindow_Reply)
 
 // Similar to ViewHostMsg_CreateWindow, except used for sub-widgets, like
 // <select> dropdowns.  This message is sent to the WebContentsImpl that
 // contains the widget being created.
 IPC_SYNC_MESSAGE_CONTROL2_2(ViewHostMsg_CreateWidget,
                             int /* opener_id */,
                             blink::WebPopupType /* popup type */,
                             int /* route_id */,
                             int32 /* surface_id */)
 
@@ skipping 535 matching lines @@
 // cached by the OS. Please see RenderMessageFilter::OnPreCacheFontCharacters
 // for details.
 IPC_SYNC_MESSAGE_CONTROL2_0(ViewHostMsg_PreCacheFontCharacters,
                             LOGFONT /* font_data */,
                             base::string16 /* characters */)
 #endif
 
 // Adding a new message? Stick to the sort order above: first platform
 // independent ViewMsg, then ifdefs for platform specific ViewMsg, then platform
 // independent ViewHostMsg, then ifdefs for platform specific ViewHostMsg.