Make window.open() IPCs be frame-based

content/browser/frame_host/render_frame_host_delegate.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_DELEGATE_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_DELEGATE_H_
 
 #include <stdint.h>
 
 #include <string>
 #include <vector>
 
 #include "base/i18n/rtl.h"
 #include "build/build_config.h"
 #include "content/browser/webui/web_ui_impl.h"
 #include "content/common/content_export.h"
 #include "content/common/frame_message_enums.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/common/javascript_message_type.h"
 #include "content/public/common/media_stream_request.h"
 #include "mojo/public/cpp/bindings/scoped_interface_endpoint_handle.h"
 #include "net/http/http_response_headers.h"
+#include "ui/base/window_open_disposition.h"
 
 #if defined(OS_WIN)
 #include "ui/gfx/native_widget_types.h"
 #endif
 
 class GURL;
 
 namespace IPC {
 class Message;
 }
 
 namespace device {
 class GeolocationServiceContext;
 class WakeLockServiceContext;
 }
 
+namespace gfx {
+class Rect;
+}
+
+namespace mojom {
+class CreateNewWindowParams;

[alexmos, 2016/12/13 18:41:40]

Is this one needed?  There's one defined below in content namespace already.

[ncarter (slow), 2016/12/15 00:33:16]

Done.

+}
+
 namespace content {
 class FrameTreeNode;
 class InterstitialPage;
 class PageState;
 class RenderFrameHost;
 class ScreenOrientationProvider;
+class SessionStorageNamespace;
 class WebContents;
 struct AXEventNotificationDetails;
 struct AXLocationChangeNotificationDetails;
 struct ContextMenuParams;
 struct FileChooserParams;
 
+namespace mojom {
+class CreateNewWindowParams;
+}
+
 // An interface implemented by an object interested in knowing about the state
 // of the RenderFrameHost.
 class CONTENT_EXPORT RenderFrameHostDelegate {
  public:
   // This is used to give the delegate a chance to filter IPC messages.
   virtual bool OnMessageReceived(RenderFrameHost* render_frame_host,
                                  const IPC::Message& message);
 
   // Allows the delegate to filter incoming associated inteface requests.
   virtual void OnAssociatedInterfaceRequest(
@@ skipping 143 matching lines @@
 
   // Set the |node| frame as focused in the current FrameTree as well as
   // possibly changing focus in distinct but related inner/outer WebContents.
   virtual void SetFocusedFrame(FrameTreeNode* node, SiteInstance* source) {}
 
   // Creates a WebUI object for a frame navigating to |url|. If no WebUI
   // applies, returns null.
   virtual std::unique_ptr<WebUIImpl> CreateWebUIForRenderFrameHost(
       const GURL& url);
 
+  // The page is trying to open a new page (e.g. a popup window). The window
+  // should be created associated with the given |route_id| in the process of

[alexmos, 2016/12/13 18:41:40]

Update |route_id|.

[ncarter (slow), 2016/12/15 00:33:16]

Done.

+  // |source_site_instance|, but it should not be shown yet. That
+  // should happen in response to ShowCreatedWindow.
+  // |params.window_container_type| describes the type of RenderViewHost
+  // container that is requested -- in particular, the window.open call may
+  // have specified 'background' and 'persistent' in the feature string.
+  //
+  // The passed |params.frame_name| parameter is the name parameter that was
+  // passed to window.open(), and will be empty if none was passed.
+  //
+  // Note: this is not called "CreateWindow" because that will clash with
+  // the Windows function which is actually a #define.
+  //
+  // The caller is expected to handle cleanup if this operation fails or is
+  // suppressed, by looking for the existence of a RenderFrameHost in
+  // source_site_instance's process with |main_frame_route_id| after this method
+  // returns.
+  virtual void CreateNewWindow(
+      SiteInstance* source_site_instance,
+      int32_t render_view_route_id,
+      int32_t main_frame_route_id,
+      int32_t main_frame_widget_route_id,
+      const mojom::CreateNewWindowParams& params,
+      SessionStorageNamespace* session_storage_namespace) {}
+
+  // Show a previously created page with the specified disposition and bounds.
+  // The window is identified by the |main_frame_widget_route_id| passed to
+  // CreateNewWindow.
+  //
+  // Note: this is not called "ShowWindow" because that will clash with
+  // the Windows function which is actually a #define.
+  virtual void ShowCreatedWindow(int process_id,
+                                 int main_frame_widget_route_id,
+                                 WindowOpenDisposition disposition,
+                                 const gfx::Rect& initial_rect,
+                                 bool user_gesture) {}
+
  protected:
   virtual ~RenderFrameHostDelegate() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_DELEGATE_H_