Closed Bug 572249 Opened 9 years ago Closed 9 years ago
flush prefs on android app pause
once paused we may not resume, might as well flash the prefs at that point
Attachment #451435 - Flags: review?(mwu)
Comment on attachment 451435 [details] [diff] [review] patch I'd remove the debugging log on the java side. Please check if SavePrefFile is ok to call from another thread.
Attachment #451435 - Flags: review?(mwu) → review+
Comment on attachment 451435 [details] [diff] [review] patch Hmm I don't really see any evidence that it's safe to call SavePrefFile from another thread. Ideally we'd fire something off to observers to tell them to prepare to exit but I haven't found something that does exactly what we want. Perhaps profile-before-change? Regardless, we'll probably have to put this on the gecko main thread through an event.
this adds a pause event and also does some shut down cleanup stuff for the close event
Attachment #452062 - Flags: review?(mwu)
Attachment #451435 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.