1529684 - Create BrowsingContext for remote frame in embedder process

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, enhancement, P1)

Description

[Nika Layzell [:nika] (ni? for response)]

This allows us to synchronously have access to this information in the embedder, and control the propagation of information better. In the future this will let us persist BrowsingContext when doing e.g. process switches, and establish links between parent and child oop iframes.

Comment 1

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1529684 - Part 1: Allow Attaching BrowsingContext from parent to child,

This is important to allow creating BrowsingContexts outside of the process
where they are going to be used. This is largely a re-arrangement of existing
code.

There is currently no way to do this type of attaching for browsing contexts in
existing BrowsingContextGroups, which creates some limitations, but happens to
be sufficient for us in the current situation.

Comment 2

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1529684 - Part 2: Create BrowsingContext for remote browsers in parent,

Depends on D21095

Comment 3

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1529684 - Part 3: Create BrowsingContext for remote iframes in embedder,

Depends on D21096

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7f2bda80e479
Part 1: Allow Attaching BrowsingContext from parent to child, r=farre
https://hg.mozilla.org/integration/autoland/rev/257d45117af3
Part 2: Create BrowsingContext for remote browsers in parent, r=farre
https://hg.mozilla.org/integration/autoland/rev/9f3e65f6b1dd
Part 3: Create BrowsingContext for remote iframes in embedder, r=farre

Comment 5

[Cristian Brindusan [:cbrindusan]]

Backed out 3 changesets (Bug 1529684) for bc crashes at [@ mozilla::dom::BrowsingContext::Detach(bool)]. (and Bug 1530550 for conflicts.)

Backout: https://hg.mozilla.org/integration/autoland/rev/1b825b6f386925ae7cbd762f6dd78a693a77a7cc

Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=pending%2Crunning%2Ctestfailed%2Cbusted%2Cexception&revision=9f3e65f6b1dda22a6e35db2c4c8e8b46377a9b93&selectedJob=231973786

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=231973786&repo=autoland&lineNumber=3084

Comment 6

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1529684 - Part 4: Add a GetContentParent method to CanonicalBrowsingContext,

Depends on D21097

Comment 7

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1529684 - Part 5: Support using remote window proxy objects in the parent process,

Depends on D22762

Comment 8

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1529684 - Part 6: Store a mIsInProcess flag to preserve WindowProxy behaviour,

Depends on D22763

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/29b316b9a035
Part 1: Allow Attaching BrowsingContext from parent to child, r=farre
https://hg.mozilla.org/integration/autoland/rev/a95dcf86a9b0
Part 2: Create BrowsingContext for remote browsers in parent, r=farre
https://hg.mozilla.org/integration/autoland/rev/6e96eaaa1b90
Part 3: Create BrowsingContext for remote iframes in embedder, r=farre
https://hg.mozilla.org/integration/autoland/rev/0a3848628be9
Part 4: Add a GetContentParent method to CanonicalBrowsingContext, r=farre
https://hg.mozilla.org/integration/autoland/rev/cb217921cf9a
Part 5: Support using remote window proxy objects in the parent process, r=farre
https://hg.mozilla.org/integration/autoland/rev/2e9f31ee6571
Part 6: Store a mIsInProcess flag to preserve WindowProxy behaviour, r=farre

Comment 10

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/29b316b9a035
https://hg.mozilla.org/mozilla-central/rev/a95dcf86a9b0
https://hg.mozilla.org/mozilla-central/rev/6e96eaaa1b90
https://hg.mozilla.org/mozilla-central/rev/0a3848628be9
https://hg.mozilla.org/mozilla-central/rev/cb217921cf9a
https://hg.mozilla.org/mozilla-central/rev/2e9f31ee6571