Remove resume_session_once pref and use nsIAppStartup.restarting and .wasRestarted

RESOLVED WONTFIX

Status

()

RESOLVED WONTFIX
5 years ago
5 years ago

People

(Reporter: ttaubert, Assigned: ttaubert)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Bug 911146 will introduce nsIAppStartup.willRestart and .wasRestarted attributes. We can simplify the SessionStore code using those new attributes and get rid of another pref.
Created attachment 801464 [details] [diff] [review]
Remove restore_session_once pref and use nsIAppStartup.willRestart and .wasRestarted attributes
Attachment #801464 - Flags: feedback?(smacleod)
(Assignee)

Updated

5 years ago
Blocks: 771065
Comment on attachment 801464 [details] [diff] [review]
Remove restore_session_once pref and use nsIAppStartup.willRestart and .wasRestarted attributes

Forgot some stuff.
Attachment #801464 - Flags: feedback?(smacleod)
(Assignee)

Updated

5 years ago
Summary: Remove restore_session_once pref and use nsIAppStartup.willRestart and .wasRestarted → Remove restore_session_once pref and use nsIAppStartup.restarting and .wasRestarted
(Assignee)

Updated

5 years ago
Component: Startup and Profile System → Session Restore
Product: Toolkit → Firefox
Darn, that seems hard to do. I completely missed the code in nsBrowserGlue.js that uses the pref to restore a session once if the user chooses to. The profile migrator also makes use of it.

Oh well, at least we now have nsIAppStartup.wasRestarted that other code might use.
No longer blocks: 771065
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
Summary: Remove restore_session_once pref and use nsIAppStartup.restarting and .wasRestarted → Remove resume_session_once pref and use nsIAppStartup.restarting and .wasRestarted
You need to log in before you can comment on or make changes to this bug.