Closed Bug 1594236 Opened 5 years ago Closed 5 years ago

Use real result principal for DocumentChannel process selection

Categories

(Core :: DOM: Content Processes, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla72
Fission Milestone M4
Tracking Status
firefox72 --- fixed

People

(Reporter: nika, Assigned: nika)

Details

Attachments

(1 file)

Currently the logic for process selection extracts a URI from the principal, and uses that URI to perform process selection. This patch adds a codepath for passing the result principal through the remote type selection logic and using it directly.

This should ideally improve the behaviour of URIs with less obvious origins, such as those which inherit their origin.

Unfortunately, OriginAttributes are still ignored by process selection, due to some code using the fallback logic which is unaware of OAs. This should be fixed in the future.

Currently the logic for process selection extracts a URI from the principal, and
uses that URI to perform process selection. This patch adds a codepath for
passing the result principal through the remote type selection logic and using
it directly.

This should ideally improve the behaviour of URIs with less obvious origins,
such as those which inherit their origin.

Unfortunately, OriginAttributes are still ignored by process selection, due to
some code using the fallback logic which is unaware of OAs. This should be fixed
in the future.

Pushed by nlayzell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/19ab4b56d218
Use real result principal for DocumentChannel process selection, r=mattwoodrow
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Assignee: nobody → nika

Retroactively moving fixed bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to an appropriate Fission Milestone.

This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:

0ee3c76a-bc79-4eb2-8d12-05dc0b68e732

Fission Milestone: --- → M4
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: