Closed Bug 1703326 Opened 1 month ago Closed 2 days ago

Stop relying on ContentSessionStore for "SessionStore:crashedTabRevived"

Categories

(Firefox :: Session Restore, task, P2)

task

Tracking

()

RESOLVED FIXED
90 Branch
Fission Milestone M7a
Tracking Status
firefox90 --- fixed

People

(Reporter: kashav, Assigned: annyG)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

Severity: -- → N/A
Fission Milestone: --- → M7a
Priority: -- → P2
Assignee: nobody → agakhokidze
Status: NEW → ASSIGNED

We can probably use chrome-page-hidden / content-page-hidden as a replacement for the "pagehide" event that we're currently relying on (see SessionStore's OBSERVING list). I'm adding a thing in https://phabricator.services.mozilla.com/D110337#C3771309NL72 for SHIP-only SessionStore notifications, so we can probably use the same thing here.

Blocks: 1709208
Attachment #9218498 - Attachment description: WIP: Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived → Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=nika!,kashav!
Attachment #9218498 - Attachment description: Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=nika!,kashav! → Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=kashav!
Attachment #9218498 - Attachment description: Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=kashav! → WIP: Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=nika!,kashav!
Attachment #9218498 - Attachment description: WIP: Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=nika!,kashav! → WIP: Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=kashav!
Attachment #9218498 - Attachment description: WIP: Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=kashav! → Bug 1703326 - Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=kashav!
Pushed by agakhokidze@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3bf273f64c48
Stop relying on ContentSessionStore for SessionStore:crashedTabRevived, r=kashav
Status: ASSIGNED → RESOLVED
Closed: 2 days ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.