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

RESOLVED FIXED

Status

()

enhancement
P2
normal
RESOLVED FIXED
7 months ago
3 months ago

People

(Reporter: mayhemer, Assigned: kershaw)

Tracking

(Blocks 1 bug)

Trunk
Points:
---

Firefox Tracking Flags

(firefox64 affected)

Details

(Whiteboard: [necko-triaged])

Attachments

(3 attachments)

(Reporter)

Updated

7 months ago
Priority: -- → P2
(Reporter)

Comment 1

7 months 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

7 months ago
Whiteboard: [necko-triaged]
(Assignee)

Comment 2

6 months ago
P3, since this project is suspended.
Priority: P2 → P3
(Assignee)

Updated

5 months ago
Priority: P3 → P2
(Assignee)

Comment 3

5 months 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

5 months ago
1. Add proxy info in HttpConnectionInfoCloneArgs
2. Add |mIPCOpen| in HttpTransactionParent
(Assignee)

Updated

5 months ago
Assignee: nobody → kershaw

Comment 5

3 months 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 6

3 months ago

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

You need to log in before you can comment on or make changes to this bug.