Give the main frame a RenderWidget.

content/renderer/render_frame_impl.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 47 matching lines @@
 class WebGeolocationClient;
 class WebMouseEvent;
 class WebContentDecryptionModule;
 class WebMediaPlayer;
 class WebPresentationClient;
 class WebPushClient;
 class WebSecurityOrigin;
 struct WebCompositionUnderline;
 struct WebContextMenuData;
 struct WebCursorInfo;
+struct WebScreenInfo;
 }
 
 namespace gfx {
 class Point;
 class Range;
 class Rect;
 }
 
 namespace media {
 class CdmFactory;
@@ skipping 43 matching lines @@
 struct ResourceResponseHead;
 struct StartNavigationParams;
 struct StreamOverrideParameters;
 class VRDispatcher;
 
 class CONTENT_EXPORT RenderFrameImpl
     : public RenderFrame,
       NON_EXPORTED_BASE(public blink::WebFrameClient),
       NON_EXPORTED_BASE(public media::WebMediaPlayerDelegate) {
  public:
-  // Creates a new RenderFrame. |render_view| is the RenderView object that this
-  // frame belongs to.
-  // Callers *must* call |SetWebFrame| immediately after creation.
-  // Note: This is called only when RenderFrame is created by Blink through
-  // createChildFrame.
-  // TODO(creis): We should structure this so that |SetWebFrame| isn't needed.
-  static RenderFrameImpl* Create(RenderViewImpl* render_view, int32 routing_id);
+  static RenderFrameImpl* CreateMainFrame(
+      RenderViewImpl* render_view,
+      int32 routing_id,
+      int32 widget_routing_id,
+      int32 surface_id,
+      bool hidden,
+      const blink::WebScreenInfo& screen_info,
+      CompositorDependencies* compositor_deps);
 
+  // TODO(dcheng): This should probably be renamed to CreateChildFrame.

[ncarter (slow), 2015/08/24 20:55:35]

I agree. There are only two callers, so I think you should just go ahead and
make the fix.

[dcheng, 2015/09/03 16:33:09]

It turns out this isn't just used for subframes, so I've deleted this TODO.

   // Creates a new RenderFrame with |routing_id| as a child of the RenderFrame
   // identified by |parent_routing_id| or as the top-level frame if the latter
   // is MSG_ROUTING_NONE. If |proxy_routing_id| is MSG_ROUTING_NONE, it creates
   // the Blink WebLocalFrame and inserts it into the frame tree after the frame
   // identified by |previous_sibling_routing_id|, or as the first child if
   // |previous_sibling_routing_id| is MSG_ROUTING_NONE. Otherwise, the frame is
   // semi-orphaned until it commits, at which point it replaces the proxy
   // identified by |proxy_routing_id|. Note: This is called only when
   // RenderFrame is being created in response to IPC message from the browser
   // process. All other frame creation is driven through Blink and Create.
@@ skipping 468 matching lines @@
     int id_;
     bool notify_result_;
     int routing_id_;
     base::WeakPtr<RenderFrameImpl> render_frame_impl_;
 
     DISALLOW_COPY_AND_ASSIGN(JavaScriptIsolatedWorldRequest);
   };
 
   typedef std::map<GURL, double> HostZoomLevels;
 
+  // Creates a new RenderFrame. |render_view| is the RenderView object that this
+  // frame belongs to.
+  // Callers *must* call |SetWebFrame| immediately after creation.
+  static RenderFrameImpl* Create(RenderViewImpl* render_view, int32 routing_id);
+
   // Functions to add and remove observers for this object.
   void AddObserver(RenderFrameObserver* observer);
   void RemoveObserver(RenderFrameObserver* observer);
 
   // Builds and sends DidCommitProvisionalLoad to the host.
   void SendDidCommitProvisionalLoad(blink::WebFrame* frame,
                                     blink::WebHistoryCommitType commit_type,
                                     const blink::WebHistoryItem& item);
 
   // IPC message handlers ------------------------------------------------------
@@ skipping 401 matching lines @@
 #endif
 
   base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_