Closed Bug 1597014 Opened 11 months ago Closed 11 months ago

Unify PDocumentChannel::RedirectToRealChannel and PContent::CrossProcessRedirect

Categories

(Core :: Networking, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: mattwoodrow, Assigned: mattwoodrow)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

These two functions have a lot of arguments that are largely the same, and mainly only differ where one is missing functionality.

They also return slightly different values, so we have to have separate promise handlers for each.

I want to create a struct for all the args, make them return the same values, and then combine the promise handling in DocumentChannelParent so that we only have one.

These two functions both have a large number of arguments that are largely identical (and ones missing from the PContent one are mostly oversights).
Unifying both to take a single argument and return the same result means we can unify the serialization and return value (promise) handling.

Priority: -- → P2
Whiteboard: [necko-triaged]
Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/09e10266c897
Unify same-process and cross-process redirect to real channel functions to avoid needed separate promise handlers. r=mayhemer
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Assignee: nobody → matt.woodrow
You need to log in before you can comment on or make changes to this bug.