Closed Bug 1500023 Opened 4 years ago Closed 4 years ago
Session File runs even when merely restoring activity
47 bytes, text/x-phabricator-request
|Details | Review|
As long we have only one GeckoApp, https://dxr.mozilla.org/mozilla-central/rev/c291143e24019097d087f9307e59b49facaf90cb/mobile/android/base/java/org/mozilla/gecko/GeckoApp.java#1205-1206 shouldn't run when we are merely restoring the activity (i.e. GeckoApp had already been active before in the same process).
Even if the GeckoApp activity is destroyed, the user's tabs remain open as long as the app process itself isn't killed. Therefore, for users who aren't automatically restoring their previous session's tabs on startup, there's no point in renaming the current session store file so it appears under "Recently closed" instead of being automatically restored, unless we're really doing a cold startup (and hence mIsRestoringActivity is false). This also better matches what we do when we *are* automatically restoring tabs: We only do it when !mIsRestoringActivity, too.
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/d6d4324ac779 Only process previous session file once per app startup. r=nalexander
You need to log in before you can comment on or make changes to this bug.