Closed Bug 1661791 Opened 4 years ago Closed 2 years ago

Remove "New Non-Fission Window" menu item and "[F]" tab tooltip suffix

Categories

(Firefox :: Menus, task, P5)

task

Tracking

()

RESOLVED FIXED
97 Branch
Tracking Status
firefox-esr91 --- wontfix
firefox95 --- wontfix
firefox96 --- wontfix
firefox97 --- fixed

People

(Reporter: cpeterson, Assigned: cpeterson)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fission])

Attachments

(2 files)

After we ship Fission to everyone, we can remove mentions of Fission from the Firefox UI, such as the "Open Non-Fission Window" menu item and "[F]" tab tooltip suffix.

We should also remove the Fission experiment pref added in bug 1660057.

Blocks: 1670821
Assignee: nobody → cpeterson
Fission Milestone: Future → ---
Summary: Remove "Open Non-Fission Window" menu item and "[F]" tab tooltip suffix → Remove "New Non-Fission Window" menu item and "[F]" tab tooltip suffix
Whiteboard: [fission]

Also, do we still need the WINDOW_REMOTE_SUBFRAMES_ENABLED_STATUS telemetry probe if Fission is the default?

(In reply to Chris Peterson [:cpeterson] from comment #1)

Nika, this bug proposes to remove the "New Non-Fission Window" menu item. Besides removing the UI, do we also want to remove support for creating non-Fission windows programmatically? That was added in bug 1542791 and is used by some tests.

I think we may want to remove support for it eventually, but I don't think we need to rush to do it right away because it'd be inconvenient for some tests. We should eventually switch it up though, probably.

When we remove the [F] marker, let's make sure to still log out the subframe pids in some way in the tab tooltip, perhaps by logging every pid in the (pid XXX) section just making sure that the toplevel pid comes first.

(In reply to Chris Peterson [:cpeterson] from comment #2)

Also, do we still need the WINDOW_REMOTE_SUBFRAMES_ENABLED_STATUS telemetry probe if Fission is the default?

Yeah, we can probably get rid of it once we've rolled out.

Flags: needinfo?(nika)

Now that Fission is the default, we don't want people to open Fission and non-Fission windows in the same session.

Instead of showing:

(pid 3) [F 2, 1, 4]

The tab tooltip will now show a combined list of pids with the content pid followed by a sorted list of subframe pids:

(pids 3, 1, 2, 4)

Depends on D133854

Pushed by cpeterson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/617d642dedfa
Remove "New Non-Fission Window" menu item. r=mconley
https://hg.mozilla.org/integration/autoland/rev/67c3b3e70953
Remove Fission [F] tab toolip and move subframe pids to the (pid) tooltip. r=mconley
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
You need to log in before you can comment on or make changes to this bug.