1617500 - Construct LoadInfo in the parent process when using DocumentChannel

Status: RESOLVED FIXED

(Core :: DOM: Navigation, task, P2)

Description

[Matt Woodrow (:mattwoodrow)]

Bug 1602318 wants to construct DocumentLoadListener directly in the parent process without a content process.

To do so, we need to be able to construct all necessary arguments using data we have in the parent.

Ideally we'll construct the LoadInfo in the parent, even for content initiated loads, so that both sources take the same code path.

This will also help with an existing bug with OOP <iframes>, where we try to construct the LoadInfo in the content process, but can't do so properly without access to our ancestors.

Comment 1

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1617500 - Don't try to compute ancestors for a top-level window in the LoadInfo constructor. r?nika

Depends on D64249

Comment 2

[Matt Woodrow (:mattwoodrow)]

Attachment: Bug 1617500 - Create top-level document LoadInfo directly in parent process with DocumentChannel. r?nika

Depends on D64641

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1a62904bfcb7
Create top-level document LoadInfo directly in parent process with DocumentChannel. r=nika

Comment 4

[Dorel Luca [:dluca]]

Backed out 7 changesets (bug 1617500, bug 1617789, bug 1617434) for build bustage in dom/clients/manager/ClientChannelHelper.cpp

Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=291144779&repo=autoland&lineNumber=28560

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&revision=1a62904bfcb7345ea90841e179e8aea5e88ffac8

Backout:
https://hg.mozilla.org/integration/autoland/rev/d9db62e5092eaeefb7049b150bdd775016c9a62d

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/13837165109d
Create top-level document LoadInfo directly in parent process with DocumentChannel. r=nika

Comment 6

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/13837165109d

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0224dccf2bb5
Don't try to compute ancestors for a top-level window in the LoadInfo constructor. r=nika,ckerschb

Comment 8

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/0224dccf2bb5