Closed
Bug 1497235
Opened 6 years ago
Closed 6 years ago
[socket-process] Pass class-of-service and request-context to http trans child
Categories
(Core :: Networking: HTTP, enhancement, P2)
Core
Networking: HTTP
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox64 | --- | affected |
People
(Reporter: mayhemer, Assigned: kershaw)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(3 obsolete files)
![]() |
Reporter | |
Updated•6 years ago
|
Priority: -- → P2
![]() |
Reporter | |
Comment 1•6 years ago
|
||
Forgot to mention that it would be good to pass this through the Init() method, if possible. Not sure we need setters (and hence also messages) at all, please check.
Updated•6 years ago
|
Whiteboard: [necko-triaged]
Assignee | ||
Updated•6 years ago
|
Priority: P3 → P2
Assignee | ||
Comment 3•6 years ago
|
||
A straightforward patch. 1. Query nsIHttpChannel and get reuqestContextID in HttpTransactionParent. 2. Pass it through the Init() method. 3. Create EnsureRequestContext again in HttpTransactionChild and pass it to the real transaction.
Assignee | ||
Comment 4•6 years ago
|
||
1. Add proxy info in HttpConnectionInfoCloneArgs 2. Add |mIPCOpen| in HttpTransactionParent
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → kershaw
Comment 5•6 years ago
|
||
We need to pass class-of-service also.
Several throttling tests are disable before.
Not sure if passing the cos works or not
https://hg.mozilla.org/projects/larch/file/322a2074e4f0/netwerk/test/unit/xpcshell.ini#l399
Don't forget to file a serializing nsProxyInfo bug
Assignee | ||
Comment 7•6 years ago
|
||
https://hg.mozilla.org/projects/larch/rev/eaf43966becc3854ecf2dae70f763fc69c281ac8
https://hg.mozilla.org/projects/larch/rev/41d3a13f5782ead12d6db9e94bf1a647f9734414
https://hg.mozilla.org/projects/larch/rev/ea6dacc7189bddb8ff3638e4eb3f8a53dd958a20
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Attachment #9027000 -
Attachment is obsolete: true
Updated•5 years ago
|
Attachment #9026999 -
Attachment is obsolete: true
Updated•3 years ago
|
Attachment #9038835 -
Attachment is obsolete: true
You need to log in
before you can comment on or make changes to this bug.
Description
•