Closed Bug 1494585 Opened 7 years ago Closed 7 years ago

Create IPC between ContentProcess and SocketProcess

Categories

(Core :: Networking: HTTP, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED

People

(Reporter: kershaw, Assigned: kershaw)

References

Details

(Whiteboard: [necko-triaged])

Attachments

(1 obsolete file)

As per bug 1484743 comment #0, we might nned to deliver data directlyt from socket process to content process. The goal of this bug is to create a bridge between content process and socket process.
Assignee: nobody → kershaw
Priority: -- → P2
Whiteboard: [necko-triaged]
Blocks: socket-proc
Summary: Create IPC between HttpChannelChild and SocketProcess → Create IPC between ContentProcess and SocketProcess
1. Introduce an empty top level IPC protocol PSocketProcessBridge 2. PSocketProcessBridgeParent lives in content process and PSocketProcessBridgeChild lives in socket process. 3. NeckoParent will try to create the bridge when |RecvPHttpChannelConstructor| is called.
Kershaw, does this patch blocks any immediate work? I want to prioritize the review, it's not a small patch. Thanks.
Flags: needinfo?(kershaw)
(In reply to Honza Bambas (:mayhemer) from comment #2) > Kershaw, does this patch blocks any immediate work? I want to prioritize > the review, it's not a small patch. Thanks. There is no need to review this for now. This patch does't block any work.
Flags: needinfo?(kershaw)
Priority: P2 → P3
Blocks: 1503834
Priority: P3 → P2
Blocks: socket-proc-webrtc
No longer blocks: socket-proc
Kershaw, can you please update the patch here according https://phabricator.services.mozilla.com/D7165#282051 (nothing major, I believe) and when the added assertions pass, land on larch? You may want to ping :bwc before landing, to sync with his local work based on larch first, though. Thanks.
Flags: needinfo?(kershaw)
Priority: P2 → P1
No longer blocks: socket-proc-webrtc
(In reply to Honza Bambas (:mayhemer) from comment #4) > Kershaw, can you please update the patch here according > https://phabricator.services.mozilla.com/D7165#282051 (nothing major, I > believe) and when the added assertions pass, land on larch? You may want to > ping :bwc before landing, to sync with his local work based on larch first, > though. > > Thanks. Patch updated.
Flags: needinfo?(kershaw)
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Attachment #9012825 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: