1947003 - Consider removing dom.noopener.newprocess.enabled pref

Status: NEW

(Core :: DOM: Navigation, task)

Description

[Gregory Pappas [:gregp]]

This pref was introduced in bug 1365032.

My understanding is that this pref is ignored when Fission is enabled. Is there still value in being able to turn it off?

Comment 1

[Gregory Pappas [:gregp]]

NI for question in comment 0

Comment 2

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

Yes, this pref is treated as-if it is disabled when Fission is enabled, so the code behind this pref is only usable when Fission is disabled (i.e. only on Android atm).

I don't think we'd want to remove the ability to turn it off, but we may want to remove the ability to turn it on in the near future, as it'd be nice to remove the corresponding mostly-dead codepath. I'd like the ability to turn it off to stick around for now in case we find a problem with the codepath guarded by this pref and need to emergency disable it on Android in a dot release or similar.