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

chrome/browser/ui/tabs/tab_strip_model.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 "chrome/browser/ui/tabs/tab_strip_model.h"
 
 #include <algorithm>
 #include <map>
 #include <string>
 
@@ skipping 686 matching lines @@
   // layout will be performed based on a width of 0 pixels, causing a
   // very long, narrow, inaccurate layout.  Because some scripts on pages (as
   // well as WebKit's anchor link location calculation) are run on the
   // initial layout and not recalculated later, we need to ensure the first
   // layout is performed with sane view dimensions even when we're opening a
   // new background tab.
   if (WebContents* old_contents = GetActiveWebContents()) {
     if ((add_types & ADD_ACTIVE) == 0) {
       contents->GetView()->SizeContents(
           old_contents->GetView()->GetContainerSize());
-      // We need to hide the contents or else we get and execute paints for
-      // background tabs. With enough background tabs they will steal the
-      // backing store of the visible tab causing flashing. See bug 20831.

[Charlie Reis, 2013/08/19 22:18:03]

Maybe add 20831 to the BUG list of the CL?  There was no test on that CL, so it
would be helpful for this commit to post a message there in case it ever
regresses.

[jamesr, 2013/08/19 23:50:40]

Done.

-      contents->WasHidden();
     }
   }
 }
 
 void TabStripModel::CloseSelectedTabs() {
   InternalCloseTabs(selection_model_.selected_indices(),
                     CLOSE_CREATE_HISTORICAL_TAB | CLOSE_USER_GESTURE);
 }
 
 void TabStripModel::SelectNextTab() {
@@ skipping 564 matching lines @@
 void TabStripModel::ForgetOpenersAndGroupsReferencing(
     const WebContents* tab) {
   for (WebContentsDataVector::const_iterator i = contents_data_.begin();
        i != contents_data_.end(); ++i) {
     if ((*i)->group == tab)
       (*i)->group = NULL;
     if ((*i)->opener == tab)
       (*i)->opener = NULL;
   }
 }