If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Remove resume_session_once pref and use nsIAppStartup.restarting and .wasRestarted

RESOLVED WONTFIX

Status

()

Firefox
Session Restore
RESOLVED WONTFIX
4 years ago
4 years ago

People

(Reporter: ttaubert, Assigned: ttaubert)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
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.
(Assignee)

Comment 1

4 years ago
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

4 years ago
Blocks: 771065
(Assignee)

Comment 2

4 years ago
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

4 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

4 years ago
Component: Startup and Profile System → Session Restore
Product: Toolkit → Firefox
(Assignee)

Comment 3

4 years ago
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: 4 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.