Closed Bug 1539163 Opened 7 months ago Closed 6 months ago

Support changing process of oop iframe during load

Categories

(Core :: DOM: Content Processes, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla68
Fission Milestone M2
Tracking Status
firefox68 --- fixed

People

(Reporter: Nika, Assigned: Nika)

References

(Blocks 1 open bug)

Details

Attachments

(3 files, 1 obsolete file)

No description provided.
Blocks: 1490803
Depends on: 1522713
Status: NEW → ASSIGNED
Depends on: 1535390, 1525427

When a remote type mismatch is found for a subframe, this patch checks if
fission is enabled for that window. If it is, it triggers a process switch,
continuing the load in a new process.

With this patch, subframes will only change process when navigating to a HTML
subframe, and not when navigating to a non-HTML subframe. That will be fixed in
a follow-up. This patch also does not change the remote type selection logic,
so only very limited types of remote iframes are supported.

Adds a basic test for this process switching functionality in subframes.

This tests one of the few forms of remoteness which are supported with the Part
1 patch, which is when navigating an iframe in a file:// document to a webpage.

This adds a codepath to the process switching logic to, rather than triggering
a fresh load, resume a process switching load.

When a remote type mismatch is found for a subframe, this patch checks if
fission is enabled for that window. If it is, it triggers a process switch,
continuing the load in a new process.

With this patch, subframes will only change process when navigating to a HTML
subframe, and not when navigating to a non-HTML subframe. That will be fixed in
a follow-up. This patch also does not change the remote type selection logic,
so only very limited types of remote iframes are supported.

Attachment #9056622 - Attachment description: Bug 1539163 - Part 2: tests for switch processes, → Bug 1539163 - Part 3: tests for switch processes,
Attachment #9056621 - Attachment is obsolete: true
Blocks: 1544811
Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0cbd910e402f
Part 1: support resuming load after process switch, r=qdot
https://hg.mozilla.org/integration/autoland/rev/578f5c0681df
Part 2: Support changing the process of subframes, r=qdot
https://hg.mozilla.org/integration/autoland/rev/3592e8434aa0
Part 3: tests for switch processes, r=qdot
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Regressions: 1563500
You need to log in before you can comment on or make changes to this bug.