1276340 - Investigate parsing the session data earlier during startup

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: Session Restore, defect, P3)

Description

[Jan Henning [:JanH]]

Once the session data parsing has been moved to a background thread in bug 905223, we should investigate whether starting the parsing slightly earlier (see proposed patch for an example) has
- any measurable (hopefully positive) effect on app startup duration?
- any unwanted side effects?

Comment 1

[Jan Henning [:JanH]]

Attachment: MozReview Request: Bug 1276340 - Move the session parsing in front of the layout initialisation.

Since we're now doing the session data parsing in background anyway (see bug 905223), we might as well start doing it slightly earlier and have it running in parallel to parts of the layout setup, so we don't bump immediately into the the following, preexisting bit of background initialisation.

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

Comment 2

[Jan Henning [:JanH]]

Attachment: MozReview Request: Bug 1276340 - DON'T LAND - Debug log

Check whether we actually have to wait for the background threads to finish their work.

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

Comment 3

[BMO Automation]

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.