1475324 - Add test for synchronous GeckoSession.saveState() usage

Status: RESOLVED FIXED

(GeckoView :: General, enhancement)

Description

[James Willcox (:snorp) ([e-mail]) (he/him)]

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 1

[James Willcox (:snorp) ([e-mail]) (he/him)]

Attachment: Bug 1475324 - Add a test for synchronous saveState()

Review commit: https://reviewboard.mozilla.org/r/256910/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/256910/

Comment 2

[Dylan Roeh (:droeh) (he/him)]

Comment on attachment 8992021 [details]
Bug 1475324 - Add a test for synchronous saveState()

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

Good idea.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e3fad7250cc3
Add a test for synchronous saveState() r=droeh

Comment 4

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/e3fad7250cc3

Comment 5

[Chris Peterson [:cpeterson]]

status-firefox62=wontfix because this is a new test so probably not worth uplifting to 62 beta.