Consider unforking sessionstore.js

RESOLVED WONTFIX

Status

defect
RESOLVED WONTFIX
6 years ago
5 years ago

People

(Reporter: Yoric, Unassigned)

Tracking

(Depends on 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: p=0)

We currently have a number implementations of Session Restore:
[1] Firefox Desktop's
[2] Fennec's
[3] Metro's.

As far as I understand, Metro's version of sessionstore.js is a fork of old Fennec's, which I seem to remember is a small subset of Desktop's. Now, we are heavily working on Desktop's to fight jank, improve I/O efficiency and, progressively, to make this e10s-friendly.

I suggest that Metro should consider adopting either Desktop's or Fennec's implementation of session restore.
No longer blocks: metrov1triage
Blocks: metrov2planning
No longer blocks: metrov1triage
No longer blocks: metrov2planning
No longer blocks: metrobacklog
Component: Browser → General
Product: Firefox for Metro → Core
Component: General → Browser
Product: Core → Firefox for Metro
Blocks: metrobacklog
Whiteboard: p=0

Comment 1

5 years ago
Is this bug valid any longer? I suspect Android will stick to its session restore implementation …
We indeed started to share some of the sessionstore modules between Desktop and Android. There's a bug about b2g doing the same. This bug is specifically about Metro so I'm not sure if we shouldn't close it, we don't work on Firefox for Metro anymore.

Comment 3

5 years ago
Yeah I thought so. Closing this bug for now. Feel free to reopen!
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.