New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 674307 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
not working at Google anymore
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Remove superfluous URL params from content::mojom::CreateNewWindowParams

Project Member Reported by nick@chromium.org, Dec 14 2016

Issue description

In the mojo struct CreateNewWindowParams, |opener_url|, |opener_security_origin|, and |opener_top_level_frame_url| are unnecessary parameters. Of these, |opener_top_level_frame_url| is particularly egregious, since an oopif isn't expected to know its top URL.

On the UI thread, we can infer their values from the state on the opener RenderFrameHost. The current pattern is problematic in part because we call FilterURL on these URLs only after they may have been used on the IO thread.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 24 2017

Comment 2 by nick@chromium.org, Apr 24 2017

Owner: nick@chromium.org
Status: Fixed (was: Available)

Sign in to add a comment