Try to restore window.opener when opening blocked popups

content/browser/web_contents/navigation_controller_impl.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 "content/browser/web_contents/navigation_controller_impl.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/strings/string_number_conversions.h"  // Temporary
@@ skipping 673 matching lines @@
           params.is_renderer_initiated,
           params.extra_headers,
           browser_context_));
   if (params.should_replace_current_entry)
     entry->set_should_replace_entry(true);
   entry->set_should_clear_history_list(params.should_clear_history_list);
   entry->SetIsOverridingUserAgent(override);
   entry->set_transferred_global_request_id(
       params.transferred_global_request_id);
   entry->SetFrameToNavigate(params.frame_name);
+  if (params.should_stay_in_site_instance) {

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

I don't think we should have this, since it makes it even more difficult to tell
what SiteInstance an entry will end up with.

[jochen (gone - plz use gerrit), 2013/07/31 18:30:52]

Removed this code.

+    entry->set_site_instance(
+        static_cast<SiteInstanceImpl*>(web_contents_->GetSiteInstance()));
+    CHECK(!entry->site_instance()->HasWrongProcessForURL(params.url));
+  }
 
   switch (params.load_type) {
     case LOAD_TYPE_DEFAULT:
       break;
     case LOAD_TYPE_BROWSER_INITIATED_HTTP_POST:
       entry->SetHasPostData(true);
       entry->SetBrowserInitiatedPostData(
           params.browser_initiated_post_data.get());
       break;
     case LOAD_TYPE_DATA:
@@ skipping 980 matching lines @@
     }
   }
 }
 
 void NavigationControllerImpl::SetGetTimestampCallbackForTest(
     const base::Callback<base::Time()>& get_timestamp_callback) {
   get_timestamp_callback_ = get_timestamp_callback;
 }
 
 }  // namespace content