Closed Bug 1617789 Opened 5 years ago Closed 5 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.

Attachment

General

Created:
Updated:
Size: