1365032 - Allow opening `noopener` links in a new process

Status: RESOLVED FIXED

(Core :: Window Management, enhancement)

Description

[Nika Layzell [:nika] (ni? for response)]

Our current noopener implementation opens the new & disconnected window in the current process. It would be nicer if we could open it in a brand new process when the current one is overloaded, or we haven't reached dom.ipc.processCount yet.

This bug tracks the changes required in order to implement this feature behind a pref.

I think I will want to wait for ehsan's changes to this code from bug 1343728 to land before reworking nsIWindowWatcher to get the APIs I need to implement this.

Comment 1

[Nika Layzell [:nika] (ni? for response)]

Attachment: Part 1: Allow changing processes when opening a new window with noopener

MozReview-Commit-ID: CXKjDxmoXVH

Comment 2

[Nika Layzell [:nika] (ni? for response)]

Attachment: Part 2: Add a test for propagating properties of windows across window.open

MozReview-Commit-ID: i7cW9rd0eO

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8874990 [details] [diff] [review]
Part 2: Add a test for propagating properties of windows across window.open

remove dumps

Comment 4

[Nika Layzell [:nika] (ni? for response)]

Attachment: Part 2: Add a test for propagating properties of windows across window.open

MozReview-Commit-ID: i7cW9rd0eO

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/77154e843ebc
Part 1: Allow changing processes when opening a new window with noopener, r=smaug
https://hg.mozilla.org/integration/mozilla-inbound/rev/7df7e414a2be
Part 2: Add a test for propagating properties of windows across window.open, r=smaug

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/77154e843ebc
https://hg.mozilla.org/mozilla-central/rev/7df7e414a2be