Open Bug 1252759 Opened 6 years ago Updated 4 years ago

[e10s] Missing support to get socket options on UDP sockets

Categories

(Core :: Networking, defect, P3)

defect

Tracking

()

Tracking Status
e10s + ---
firefox47 --- affected

People

(Reporter: drno, Unassigned)

Details

(Whiteboard: [necko-backlog])

nsUDPSocket::SetSocketOption() exists, but the counter part nsUDPSocket::GetSocketOption() is missing. As a result a couple of the nsUDPSocket::Get*() function just return NS_ERROR_NOT_IMPLEMENTED. It would be helpful to be able to query the socket options like one can today without e10s via PR_GetSocketOption() calls.
Whiteboard: [necko-backlog]
What kind of functional regression does this cause, when we ship e10s? That is what informs the e10s-blocking decision.
Flags: needinfo?(drno)
(In reply to Benjamin Smedberg  [:bsmedberg] from comment #1)
> What kind of functional regression does this cause, when we ship e10s? That
> is what informs the e10s-blocking decision.

Right now it is not blocking anything as we don't use it (yet). So for now it's the "nice to have" category. But I'm working on bug 1217677 and bug 1251821 where being able to query the buffer size before setting (to ensure that we are in fact increasing it, rather then decreasing like here: https://dxr.mozilla.org/mozilla-central/source/netwerk/base/nsSocketTransport2.cpp#1695) it might be helpful.
Flags: needinfo?(drno)
tracking-e10s: --- → +
Priority: -- → P3
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P3 → P1
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: P1 → P3
You need to log in before you can comment on or make changes to this bug.