Move all uses of UDPSocket to PBackground

NEW
Unassigned

Status

()

P3
normal
4 years ago
2 months ago

People

(Reporter: jesup, Unassigned)

Tracking

(Depends on: 1 bug, {perf})

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox41 affected)

Details

(Whiteboard: [necko-backlog])

(Reporter)

Description

4 years ago
UDPSocket currently uses PNecko, which means that all traffic touches MainThread.  This can be hugely delay-inducing, such as when I saw 3-second+ RTT for a loopback webrtc call (because MainThread on B2G was busy).

It should use PBackground.  mtransport (i.e. webrtc) is moving to PBackground in bug 1109338.  That includes much of the work needed to move other uses of UDPSocket to PBackground; the only bit left would be to extend the non-NULL PrincipalInfo support using the WIP patch I'll attach here.  The remaining part of that is to block Bind from completing until Principal checks have completed.
Whiteboard: [necko-backlog]

Comment 3

2 months ago
When we do this, we should validate that the principal supplied is valid for the content process it comes from.
Depends on: 1491018
You need to log in before you can comment on or make changes to this bug.