Closed Bug 572249 Opened 9 years ago Closed 9 years ago

flush prefs on android app pause

Categories

(Core :: Widget: Android, defect)

ARM
Android
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: blassey, Assigned: blassey)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) — Splinter Review
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+
Attachment #451435 - Flags: 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.
Attached patch patchSplinter Review
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
Attachment #452062 - Flags: review?(mwu) → review+
pushed http://hg.mozilla.org/mozilla-central/rev/5365d2057b91
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee: nobody → blassey.bugs
You need to log in before you can comment on or make changes to this bug.