Closed Bug 1544811 Opened 6 years ago Closed 6 years ago

Use process-per-site when in a fission-enabled window

Categories

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

defect

Tracking

()

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

People

(Reporter: nika, Assigned: nika)

References

Details

Attachments

(1 file)

We should change the logic to perform process switches on a per-site basis when within a fission-aware window. This will allow us to get an initial implementation of process-per-origin.

This patch introduces a new type of content process, which has a dynamic name.
This type of content process is labeled as webIsolated=${SITE_ORIGIN} and is
used within fission-enabled windows.

To enable this, additional information about the fission status of the target
window must be passed into E10SUtils. This was done by updating every call site
manually to pass an extra boolean. A better solution perhaps should be used in
the future.

With this patch enabled, we now perform process switches, but only when
navigating to HTTP URIs. If we navigate to a non-HTTP URI in an iframe with
fission enabled, it will not behave correctly. This must be done in a
follow-up.

Whiteboard: [5/3] waiting for review
Pushed by nlayzell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7f0dd201e059 Use web processes on a per-site basis for fission-enabled windows, r=mconley
Whiteboard: [5/3] waiting for review
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Regressions: 1549580
Regressions: 1561133
Regressions: 1562059
Regressions: 1572887
Regressions: 1582091
Regressions: 1587150
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: