Initialize RenderWidget(Host)(View)s with correct visibility state

content/renderer/render_widget.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.
 
 #ifndef CONTENT_RENDERER_RENDER_WIDGET_H_
 #define CONTENT_RENDERER_RENDER_WIDGET_H_
 
 #include <deque>
 #include <map>
 
@@ skipping 209 matching lines @@
   // For unit tests.
   friend class RenderWidgetTest;
 
   enum ResizeAck {
     SEND_RESIZE_ACK,
     NO_RESIZE_ACK,
   };
 
   RenderWidget(WebKit::WebPopupType popup_type,
                const WebKit::WebScreenInfo& screen_info,
-               bool swapped_out);
+               bool swapped_out,
+               bool hidden);

[sky, 2013/08/19 20:21:47]

From a readability perspective I prefer this to be an enum, so that call sites
look like:

new RenderWidget(...RenderWidget::VISIBILE), but that isn't in any style guide
so I leave it to you have if you want an enum here.

 
   virtual ~RenderWidget();
 
   // Initializes this view with the given opener.  CompleteInit must be called
   // later.
   bool Init(int32 opener_id);
 
   // Called by Init and subclasses to perform initialization.
   bool DoInit(int32 opener_id,
               WebKit::WebWidget* web_widget,
@@ skipping 495 matching lines @@
 #endif
 
   base::WeakPtrFactory<RenderWidget> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidget);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_RENDER_WIDGET_H_