Pass the size to the RenderView on creation.

content/renderer/render_view_impl.h

Index: content/renderer/render_view_impl.h
diff --git a/content/renderer/render_view_impl.h b/content/renderer/render_view_impl.h
index 67adc19750321faa4bbb449b288f39b22fdf7cf5..8b33ce07abf6a12536c8ff66664ef74da46e03b9 100644
--- a/content/renderer/render_view_impl.h
+++ b/content/renderer/render_view_impl.h
@@ -78,6 +78,7 @@ class SkBitmap;
 struct PP_NetAddress_Private;
 struct FrameMsg_Navigate_Params;
 struct ViewMsg_PostMessage_Params;
+struct ViewMsg_Resize_Params;
 struct ViewMsg_StopFinding_Params;
 
 namespace base {
@@ -178,7 +179,8 @@ class CONTENT_EXPORT RenderViewImpl
                                 bool hidden,
                                 bool never_visible,
                                 int32 next_page_id,
-                                const blink::WebScreenInfo& screen_info);
+                                const blink::WebScreenInfo& screen_info,
+                                const ViewMsg_Resize_Params* initial_size);

[piman, 2014/10/29 18:42:57]

I have a preference towards always passing a ViewMsg_Resize_Params (by const
ref), that includes the screen_info. It's more consistent and a bit simpler...
The default values look identical too (between the ViewMsg_Resize_Params on one
hand, and the various classes that hold the resulting values on the other hand).

[mkosiba (inactive), 2014/10/31 14:35:06]

The reason I was trying to pass this by pointer is to make it optional. In the
popup window case we pass null here.

I'll change the popup window case to create a ViewMsg_Resize_Params and set its
screen info since the RenderWidget::OnResize code looks like it should handle
the empty size case fine.

 
   // Used by content_layouttest_support to hook into the creation of
   // RenderViewImpls.