Make it so that allow_js_access: false can be used with background pages created by window.open.

content/browser/renderer_host/render_widget_helper.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/renderer_host/render_widget_helper.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/eintr_wrapper.h"
 #include "base/threading/thread.h"
@@ skipping 196 matching lines @@
       render_process_id_, render_widget_id);
 }
 
 void RenderWidgetHelper::OnCrossSiteSwapOutACK(
     const ViewMsg_SwapOut_Params& params) {
   resource_dispatcher_host_->OnSwapOutACK(params);
 }
 
 void RenderWidgetHelper::CreateNewWindow(
     const ViewHostMsg_CreateWindow_Params& params,
+    content::ContentBrowserClient::CanCreateWindowResult can_create_result,
     base::ProcessHandle render_process,
     int* route_id,
     int* surface_id) {
-  if (params.opener_suppressed) {
-    // If the opener is supppressed, we should open the window in a new
-    // BrowsingInstance, and thus a new process.  That means the current
-    // renderer process will not be able to route messages to it.  Because of
-    // this, we will immediately show and navigate the window in
-    // OnCreateWindowOnUI, using the params provided here.
+  bool no_js_access = can_create_result ==
+      content::ContentBrowserClient::CAN_CREATE_WINDOW_NO_JS_ACCESS;
+  if (params.opener_suppressed || no_js_access) {
+    // If the opener is supppressed or script access is disallowed, we should
+    // open the window in a new BrowsingInstance, and thus a new process. That
+    // means the current renderer process will not be able to route messages to
+    // it. Because of this, we will immediately show and navigate the window
+    // in OnCreateWindowOnUI, using the params provided here.
     *route_id = MSG_ROUTING_NONE;
     *surface_id = 0;
   } else {
     *route_id = GetNextRoutingID();
     *surface_id = GpuSurfaceTracker::Get()->AddSurfaceForRenderer(
         render_process_id_, *route_id);
     // Block resource requests until the view is created, since the HWND might
     // be needed if a response ends up creating a plugin.
     resource_dispatcher_host_->BlockRequestsForRoute(
         render_process_id_, *route_id);
@@ skipping 119 matching lines @@
 void RenderWidgetHelper::ClearAllocatedDIBs() {
   for (std::map<TransportDIB::Id, int>::iterator
        i = allocated_dibs_.begin(); i != allocated_dibs_.end(); ++i) {
     if (HANDLE_EINTR(close(i->second)) < 0)
       PLOG(ERROR) << "close: " << i->first;
   }
 
   allocated_dibs_.clear();
 }
 #endif