Closed
Bug 1494585
Opened 7 years ago
Closed 7 years ago
Create IPC between ContentProcess and SocketProcess
Categories
(Core :: Networking: HTTP, enhancement, P1)
Core
Networking: HTTP
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 | ||
Updated•7 years ago
|
Assignee: nobody → kershaw
Priority: -- → P2
Whiteboard: [necko-triaged]
Assignee | ||
Updated•7 years ago
|
Blocks: socket-proc
Assignee | ||
Updated•7 years ago
|
Summary: Create IPC between HttpChannelChild and SocketProcess → Create IPC between ContentProcess and SocketProcess
Assignee | ||
Comment 1•7 years ago
|
||
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.
![]() |
||
Comment 2•7 years ago
|
||
Kershaw, does this patch blocks any immediate work? I want to prioritize the review, it's not a small patch. Thanks.
Flags: needinfo?(kershaw)
Assignee | ||
Comment 3•7 years ago
|
||
(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)
Assignee | ||
Updated•7 years ago
|
Priority: P2 → P3
![]() |
||
Updated•7 years ago
|
Priority: P3 → P2
![]() |
||
Updated•7 years ago
|
![]() |
||
Comment 4•7 years ago
|
||
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
![]() |
||
Updated•7 years ago
|
No longer blocks: socket-proc-webrtc
![]() |
||
Updated•7 years ago
|
Blocks: socket-proc-webrtc
Assignee | ||
Comment 5•7 years ago
|
||
(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)
Assignee | ||
Comment 6•7 years ago
|
||
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Attachment #9012825 -
Attachment is obsolete: true
You need to log in
before you can comment on or make changes to this bug.
Description
•