Remove opener URLs from CreateNewWindowParams.

content/common/frame.mojom

 // Copyright 2014 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.
 
 module content.mojom;
 
 import "content/public/common/window_container_type.mojom";
 import "services/service_manager/public/interfaces/interface_provider.mojom";
 import "third_party/WebKit/public/platform/referrer.mojom";
 import "third_party/WebKit/public/web/window_features.mojom";
@@ skipping 23 matching lines @@
 interface FrameHostInterfaceBroker {
   GetInterfaceProvider(service_manager.mojom.InterfaceProvider& interfaces);
 };
 
 // Implemented by a service that provides implementations of the Frame
 // interface. (e.g. renderer processes).
 interface FrameFactory {
   CreateFrame(int32 frame_routing_id, Frame& frame, FrameHostInterfaceBroker host);
 };
 
-// TODO(csharrison): Clean up this struct. Some of the entries (like
-// opener_top_level_frame_url) are better suited to be computed in the browser
-// process. See http://crbug.com/674307.
 struct CreateNewWindowParams {
   // True if this open request came in the context of a user gesture.
   bool user_gesture;
 
   // Type of window requested.
   WindowContainerType window_container_type;
 
   // The session storage namespace ID this window should use.
   int64 session_storage_namespace_id;
 
   // The name of the resulting frame that should be created (empty if none
   // has been specified). UTF8 encoded string.
   string frame_name;
 
-  // The URL of the frame initiating the open.
-  url.mojom.Url opener_url;
-
-  // The URL of the top frame containing the opener.
-  url.mojom.Url opener_top_level_frame_url;
-
-  // The security origin of the frame initiating the open.
-  url.mojom.Url opener_security_origin;
-
   // Whether the opener will be suppressed in the new window, in which case
   // scripting the new window is not allowed.
   bool opener_suppressed;
 
   // Whether the window should be opened in the foreground, background, etc.
   ui.mojom.WindowOpenDisposition disposition;
 
   // The URL that will be loaded in the new window (empty if none has been
   // specified).
   url.mojom.Url target_url;
@@ skipping 26 matching lines @@
 
 // Implemented by the frame server (i.e. the browser process). For messages that
 // must be associated with the IPC channel.
 interface FrameHost {
   // Sent by the renderer when it is creating a new window.  The browser creates
   // a tab for it.  If |reply.route_id| is MSG_ROUTING_NONE, the window couldn't
   // be created.
   [Sync] CreateNewWindow(CreateNewWindowParams params)
       => (CreateNewWindowReply reply);
 };