As part of the Fission work we need to migrate all our framescripts to Actors. This tracks the work needed to move SessionStateAggregator.js to a child actor.

Looks like platform is making this easier for us so let's wait for that.

This also includes enabling the following tests on Fission:

HistoryDelegateTest#onHistoryStateChangeSavingState
ProgressDelegateTest#saveAndRestoreStateNewSession
ProgressDelegateTest#saveAndRestoreState
ProgressDelegateTest#flushSessionState
ProgressDelegateTest#noHistoryDelegateOnSessionStateChange
NavigationDelegateTest#purgeHistory

New test that relies on SessionStateStorage:

WebExtensionTest#testRestoringExtensionPagePreservesMessages

from Bug 1675644.

Discussed in the Fission-Android meeting, sounds like it might take a while for the desktop code to migrate so maybe we want to have a stop-gap migration first.

Tracking this bug for Android Fission milestone M2 (pass tests with Fission enabled)

Email from Andreas Farre [:farre]:

I've been digging around a bit in SessionStateAggregator.js to see
what's needed to move it to the parent process in preparation for
Fission. Have you a plan in mind for how you want to do this?

With the work we've already done it feels like collection should be
fairly easy to port to GeckoView, using code from TabStateCache.jsm[1]
to handle incremental updates from different content children (or
really different WindowGlobalChild) as some kind of intermediary step
before doing SessionState.updateSessionState. Or something similar.

For the restore step I'll have to bring in Kashav for input, since
he's the more knowledgeable. My rough overview seems to indicate that
restore might be harder. This is also more dependant on Session
History in the parent process, I guess.

In the end we want to have a system that is on, even if Fission isn't.

This is a bit cursory, but I thought that it would at least help us get started!

Agi, what's the expected timescale for this bug? This blocks bug 1758481 which we would like to complete in the 103 cycle (not before to avoid breaking some things just before ESR), would you be able to do this in that timeframe?

Hi Mark, Sorry I didn't realize this was time sensitive, we probably will get this done by 103 but I don't see any reason to delay the other patch in the meanwhile. I approved https://phabricator.services.mozilla.com/D142757

Thank you.

Cathy will pick up Agi's WIP after she lands bug 1734394.

Moving Android Fission bugs to the new GeckoView::Sandboxing component.

105

https://hg.mozilla.org/mozilla-central/rev/8dbbc6a6b494