Try to restore window.opener when opening blocked popups

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 19 matching lines @@
 #include "chrome/browser/ui/singleton_tabs.h"
 #include "chrome/browser/ui/status_bubble.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/browser/web_applications/web_app.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/browser_url_handler.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/render_view_host.h"
+#include "content/public/browser/site_instance.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_view.h"
 
 #if defined(USE_AURA)
 #include "ui/aura/window.h"
 #endif
 
 using content::GlobalRequestID;
 using content::WebContents;
 
@@ skipping 260 matching lines @@
   chrome::NavigateParams* params_;
   scoped_ptr<WebContents> target_contents_owner_;
   DISALLOW_COPY_AND_ASSIGN(ScopedTargetContentsOwner);
 };
 
 content::WebContents* CreateTargetContents(const chrome::NavigateParams& params,
                                            const GURL& url) {
   WebContents::CreateParams create_params(
       params.browser->profile(),
       tab_util::GetSiteInstanceForNewTab(params.browser->profile(), url));
+
+  // If we want to set the opener, we need to use the same SiteInstance.

[Charlie Reis, 2013/07/30 01:26:30]

I'm skeptical about this, since it shouldn't always be the same SiteInstance. 
We have careful logic in WebContentsImpl::CreateNewWindow about which
SiteInstance to use, and this approach loses that.  We probably need to find a
better way to handle this.

[jochen (gone - plz use gerrit), 2013/07/30 14:50:39]

The popup needs to be in the same process as the opener, otherwise it can't
access window.opener.

In fact, setting the site_instance here is a no-op:
NavigationController::LoadURL will create a NavigationEntry with a NULL site
instance anyway, and so
RenderViewHostController::PrepareRenderViewHostForNavigation creates a new one
if the URL differs from the site url.

[Charlie Reis, 2013/07/31 17:16:16]

That's not true.  Creating opener RVHs allows it to access a placeholder
window.opener when it's cross-site.
What is RenderViewHostController::PrepareRenderViewHostForNavigation?  Do you
mean RenderViewHostManager::UpdateRendererStateForNavigate?  If we're going
through that, we'll have the opposite problem-- creating a new SiteInstance more
frequently than we should.  Most cross-site window.open calls should stay in the
same SiteInstance-- only special cases like rel=noreferrer or the new
site-per-process logic should swap.

+  if (params.source_contents && params.should_set_opener)
+    create_params.site_instance = params.source_contents->GetSiteInstance();
   if (params.source_contents) {
     create_params.initial_size =
         params.source_contents->GetView()->GetContainerSize();
   }
+  if (params.source_contents && params.should_copy_session_storage_namespace) {
+    // To clone the session storage namespace, we need a site instance.
+    if (!create_params.site_instance) {
+      create_params.site_instance =
+          content::SiteInstance::CreateForURL(params.browser->profile(), url);
+    }
+  }
 #if defined(USE_AURA)
   if (params.browser->window() &&
       params.browser->window()->GetNativeWindow()) {
     create_params.context =
         params.browser->window()->GetNativeWindow();
   }
 #endif
+  WebContents* target_contents;
+  if (params.source_contents &&
+      (params.should_set_opener ||
+       params.should_copy_session_storage_namespace)) {
+    DCHECK(!params.should_set_opener ||

[Bernhard Bauer, 2013/07/30 08:34:50]

Wait, you have here

  if (a || b) {
    DCHECK(!a || b);
  }

Is that really the behavior you want?

+           params.should_copy_session_storage_namespace);
+    target_contents = WebContents::CreateWithOpener(
+        create_params, params.source_contents, params.should_set_opener);
+  } else {
+    target_contents = WebContents::Create(create_params);
+  }
 
-  content::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
   // failures in startup tests.
   // By default, content believes it is not hidden.  When adding contents
   // in the background, tell it that it's hidden.
@@ skipping 43 matching lines @@
       is_renderer_initiated(false),
       tabstrip_index(-1),
       tabstrip_add_types(TabStripModel::ADD_ACTIVE),
       window_action(NO_ACTION),
       user_gesture(true),
       path_behavior(RESPECT),
       ref_behavior(IGNORE_REF),
       browser(a_browser),
       initiating_profile(NULL),
       host_desktop_type(GetHostDesktop(a_browser)),
-      should_replace_current_entry(false) {
+      should_replace_current_entry(false),
+      should_copy_session_storage_namespace(false),
+      should_set_opener(false) {
 }
 
 NavigateParams::NavigateParams(Browser* a_browser,
                                WebContents* a_target_contents)
     : target_contents(a_target_contents),
       source_contents(NULL),
       disposition(CURRENT_TAB),
       transition(content::PAGE_TRANSITION_LINK),
       is_renderer_initiated(false),
       tabstrip_index(-1),
       tabstrip_add_types(TabStripModel::ADD_ACTIVE),
       window_action(NO_ACTION),
       user_gesture(true),
       path_behavior(RESPECT),
       ref_behavior(IGNORE_REF),
       browser(a_browser),
       initiating_profile(NULL),
       host_desktop_type(GetHostDesktop(a_browser)),
-      should_replace_current_entry(false) {
+      should_replace_current_entry(false),
+      should_copy_session_storage_namespace(false),
+      should_set_opener(false) {
 }
 
 NavigateParams::NavigateParams(Profile* a_profile,
                                const GURL& a_url,
                                content::PageTransition a_transition)
     : url(a_url),
       target_contents(NULL),
       source_contents(NULL),
       disposition(NEW_FOREGROUND_TAB),
       transition(a_transition),
       is_renderer_initiated(false),
       tabstrip_index(-1),
       tabstrip_add_types(TabStripModel::ADD_ACTIVE),
       window_action(SHOW_WINDOW),
       user_gesture(true),
       path_behavior(RESPECT),
       ref_behavior(IGNORE_REF),
       browser(NULL),
       initiating_profile(a_profile),
       host_desktop_type(chrome::GetActiveDesktop()),
-      should_replace_current_entry(false) {
+      should_replace_current_entry(false),
+      should_copy_session_storage_namespace(false),
+      should_set_opener(false) {
 }
 
 NavigateParams::~NavigateParams() {}
 
 void FillNavigateParamsFromOpenURLParams(chrome::NavigateParams* nav_params,
                                          const content::OpenURLParams& params) {
   nav_params->referrer = params.referrer;
   nav_params->extra_headers = params.extra_headers;
   nav_params->disposition = params.disposition;
   nav_params->override_encoding = params.override_encoding;
@@ skipping 223 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