Closed Bug 1522641 Opened 5 years ago Closed 5 years ago

Perform process switches immediately before calling OnStartRequest for HTTP(s) channels

Categories

(Core :: Networking, enhancement, P2)

Product:
Core
Component:
Networking
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Project Flags:
Fission Milestone M5

People

(Reporter: nika, Unassigned)

References

(Depends on 1 open bug)

Details

(Whiteboard: [necko-triaged])

In bug 1467223, the interception point used for performing process switches is during http-on-examine-response. Unfortunately, there is no way during this observer notification to detect whether another redirect will occur. This means that process swaps may be triggered multiple times during the load as each intermediate resource is loaded.

We should instead perform these switches on HTTP channels immediately before calling OnStartRequest on the listener, such that all other redirects have already been handled.

Whiteboard: [necko-triaged]
Blocks: 1490803
No longer blocks: fission
Depends on: document-channel
Blocks: document-channel
No longer blocks: 1490803
Fission Milestone: --- → M4
No longer depends on: document-channel
No longer blocks: document-channel

Nhi, since Valentin is on PTO, could you please find another assignee for this?

Flags: needinfo?(nhnguyen)
Assignee: valentin.gosu → juhsu
Flags: needinfo?(nhnguyen)
Depends on: document-channel

Let's wait for document channel

Assignee: juhsu → nobody

Roll some unfixed bugs from Fission Milestone M4 to M5

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

Fission Milestone: M4 → M5

This was fixed by DocumentChannel itself.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.