Closed
Bug 1527081
Opened 6 years ago
Closed 6 years ago
Don't create preloaded "new tab" browsers purely based on session restore creating new tabs for every window
Categories
(Firefox :: Tabbed Browser, enhancement, P1)
Firefox
Tabbed Browser
Tracking
()
RESOLVED
FIXED
Firefox 67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: Gijs, Assigned: Gijs)
References
(Blocks 1 open bug)
Details
(Keywords: memory-footprint, perf, Whiteboard: [fxperf:p1])
Attachments
(1 file)
Our preload browser creation seems like it could be improved.
Specifically, it currently creates preload tabs when a new tab finishes opening. There are two immediately-obvious downsides here:
- the first new tab opened on non-session-restored sessions (which is most of our users' sessions!) will never be preloaded
- for sessions restored that have a bunch of windows (with more than 1 tab per window), we'll create as many preloaded tabs, also when those windows are in the background and in some cases obviously minutes / hours / years before they even get focus. That wastes a bunch of cpu and memory.
This bug will try to address (2). I'm open to ideas for (1) if it's obvious how else we should be approaching that problem.
Assignee | ||
Comment 1•6 years ago
|
||
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/7703d1ff182f
don't create preloaded tabs for session-restored additional tabs, r=dao
Updated•6 years ago
|
Priority: -- → P1
Comment 3•6 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 67
Assignee | ||
Updated•6 years ago
|
Whiteboard: [fxperf] → [fxperf:p1]
Updated•6 years ago
|
Keywords: memory-footprint,
perf
You need to log in
before you can comment on or make changes to this bug.
Description
•