1497235 - [socket-process] Pass class-of-service and request-context to http trans child

Status: RESOLVED FIXED

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

Description

[Honza Bambas (:mayhemer)]

https://hg.mozilla.org/projects/larch/file/322a2074e4f0/netwerk/protocol/http/nsHttpChannel.cpp#l1440

Comment 1

[Honza Bambas (:mayhemer)]

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.

Comment 2

[Kershaw Chang [:kershaw]]

P3, since this project is suspended.

Comment 3

[Kershaw Chang [:kershaw]]

Attachment: Bug 1497235 - P1: Pass request context id to HttpTransactionChild

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.

Comment 4

[Kershaw Chang [:kershaw]]

Attachment: Bug 1497235 - P2: Pass ProxyInfo to socket process

1. Add proxy info in HttpConnectionInfoCloneArgs
2. Add |mIPCOpen| in HttpTransactionParent

Comment 5

[Junior [inactive]]

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

Comment 6

[Kershaw Chang [:kershaw]]

Attachment: Bug 1497235 - P3: Pass classOfService to socket process

Simply pass classOfService to socket process via Init() message.

Comment 7

[Kershaw Chang [:kershaw]]

https://hg.mozilla.org/projects/larch/rev/eaf43966becc3854ecf2dae70f763fc69c281ac8
https://hg.mozilla.org/projects/larch/rev/41d3a13f5782ead12d6db9e94bf1a647f9734414
https://hg.mozilla.org/projects/larch/rev/ea6dacc7189bddb8ff3638e4eb3f8a53dd958a20