Closed Bug 1728909 Opened 3 years ago Closed 2 years ago

Pinned tabs created by session restore should get their URL passed with skipLoad: true, instead of not passing a URL

Categories

(Firefox :: Session Restore, defect)

Product:
Firefox
Component:
Session Restore
Platform:
Desktop
All
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Project Flags:
Performance Impact low
Tracking Flags:
Tracking Status
firefox93 --- affected

People

(Reporter: Gijs, Unassigned)

References

Details

(Keywords: perf:frontend, perf:startup)

Passing the URL would allow us a chance to pick the right process, also for fission, for the new browser we're creating. As it is, we always have to process switch with fission enabled.

See Also: → 1728910
Whiteboard: [fxperf] → [fxperf:p1]
Whiteboard: [fxperf:p1] → [fxperf:p2]
Performance Impact: --- → ?
Keywords: perf:frontend, perf:startup
Whiteboard: [fxperf:p2]
Performance Impact: ? → P3

I think this got fixed in https://phabricator.services.mozilla.com/D141146 / bug 1728332 - Nika, can you confirm if there's something left to do here?

Flags: needinfo?(nika)

(Some of the see also bugs might also be fixed?)

It's not quite changed in the suggested way (non-lazy tabs still are created without their URI), but the process selection is performed in the caller now so the main goal of this bug is fixed. I think we can probably close the bug.

Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(nika)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.