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

chrome/browser/ui/browser_navigator.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/browser_navigator.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/prefs/pref_service.h"
@@ skipping 314 matching lines @@
                                            const GURL& url) {
   WebContents::CreateParams create_params(
       params.browser->profile(),
       tab_util::GetSiteInstanceForNewTab(params.browser->profile(), url));
   if (params.source_contents) {
     create_params.initial_size =
         params.source_contents->GetView()->GetContainerSize();
     if (params.should_set_opener)
       create_params.opener = params.source_contents;
   }
+  if (params.disposition == NEW_BACKGROUND_TAB)
+    create_params.initially_hidden = true;
+
 #if defined(USE_AURA)
   if (params.browser->window() &&
       params.browser->window()->GetNativeWindow()) {
     create_params.context =
         params.browser->window()->GetNativeWindow();
   }
 #endif
 
   WebContents* target_contents = WebContents::Create(create_params);
 
   // New tabs can have WebUI URLs that will make calls back to arbitrary
   // tab helpers, so the entire set of tab helpers needs to be set up
   // immediately.
   BrowserNavigatorWebContentsAdoption::AttachTabHelpers(target_contents);
   extensions::TabHelper::FromWebContents(target_contents)->
       SetExtensionAppById(params.extension_app_id);
   // TODO(sky): Figure out why this is needed. Without it we seem to get

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

I wonder if this can be nuked now (don't feel you have to investigate this).

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

I believe it can - I traced this through and it doesn't actually do anything
since the RenderWidgetHostView isn't made yet.

   // failures in startup tests.
   // By default, content believes it is not hidden.  When adding contents
   // in the background, tell it that it's hidden.
   if ((params.tabstrip_add_types & TabStripModel::ADD_ACTIVE) == 0) {
     // TabStripModel::AddWebContents invokes WasHidden if not foreground.
     target_contents->WasHidden();
   }
   return target_contents;
 }
 
@@ skipping 328 matching lines @@
   bool reverse_on_redirect = false;
   content::BrowserURLHandler::GetInstance()->RewriteURLIfNecessary(
       &rewritten_url, browser_context, &reverse_on_redirect);
 
   // Some URLs are mapped to uber subpages. Do not allow them in incognito.
   return !(rewritten_url.scheme() == chrome::kChromeUIScheme &&
            rewritten_url.host() == chrome::kChromeUIUberHost);
 }
 
 }  // namespace chrome