Closed Bug 1617789 Opened 3 years ago Closed 3 years ago

Move mutation of LoadInfo for DocumentChannel into the parent process

Categories

(Core :: DOM: Networking, task)

task
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla75
Tracking Status
firefox75 --- fixed

People

(Reporter: mattwoodrow, Assigned: mattwoodrow)

References

Details

Attachments

(4 files)

In bug 1617500, I want to try constructing LoadInfo entirely in the parent process with DocumentChannel, so that we can have a single path for both docshell-initiated loads as well as CanonicalBrowsingContext-initiated loads.

We currently have a bunch of code that configures our LoadInfo using the nsDocShellLoadState, and we can move this into CreateAndConfigureRealChannelForLoadState so that it runs in the parent process with DocumentChannel.

Depends on D63995

We currently extract Document pointers from the old LoadInfo, and attach them to the new one, such that they aren't lost during serialization.
This moves us to setting them more directly from their original source, since the old LoadInfo is no longer reliable (and was only configured fully in the parent process).

Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/98ea95f7fe87
Don't require loadInfo for navigate-to. r=nika
https://hg.mozilla.org/integration/autoland/rev/f3518664d654
Simplify load from external. r=nika
https://hg.mozilla.org/integration/autoland/rev/995edb2e7f98
Move loadInfo configuration into CreateAndConfigureRealChannelForLoadState so that it runs in the parent process with DocumentChannel. r=nika
https://hg.mozilla.org/integration/autoland/rev/592a5ddf6acf
Set content process Document pointer without relying on LoadInfo. r=nika
Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/269def105275
Don't require loadInfo for navigate-to. r=nika
https://hg.mozilla.org/integration/autoland/rev/8b2f20cab5e9
Simplify load from external. r=nika
https://hg.mozilla.org/integration/autoland/rev/ab7787929c48
Move loadInfo configuration into CreateAndConfigureRealChannelForLoadState so that it runs in the parent process with DocumentChannel. r=nika
https://hg.mozilla.org/integration/autoland/rev/437da2aca7a0
Set content process Document pointer without relying on LoadInfo. r=nika
Flags: needinfo?(matt.woodrow)
You need to log in before you can comment on or make changes to this bug.