Add test for synchronous GeckoSession.saveState() usage

RESOLVED FIXED in Firefox 63

Status

enhancement
RESOLVED FIXED
11 months ago
5 months ago

People

(Reporter: snorp, Assigned: snorp)

Tracking

unspecified
mozilla63
Unspecified
Android

Firefox Tracking Flags

(firefox-esr52 wontfix, firefox-esr60 wontfix, firefox61 wontfix, firefox62 wontfix, firefox63 fixed)

Details

Attachments

(1 attachment)

We know apps sometimes want to block in the UI thread for a saveState() result, because they are in onStop() or onPause() and want to make sure they are able to restore things later. We should add a test to make sure this works as expected.

Comment 2

10 months ago
mozreview-review
Comment on attachment 8992021 [details]
Bug 1475324 - Add a test for synchronous saveState()

https://reviewboard.mozilla.org/r/256910/#review263828

Good idea.
Attachment #8992021 - Flags: review?(droeh) → review+

Comment 3

10 months ago
Pushed by jwillcox@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e3fad7250cc3
Add a test for synchronous saveState() r=droeh

Comment 4

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e3fad7250cc3
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Assignee: nobody → snorp
Version: Firefox 59 → unspecified
status-firefox62=wontfix because this is a new test so probably not worth uplifting to 62 beta.
OS: Unspecified → Android

Updated

5 months ago
Product: Firefox for Android → GeckoView
Target Milestone: Firefox 63 → mozilla63
You need to log in before you can comment on or make changes to this bug.