1176382 - [Meta] Tracking for ICE TCP user pref to be turned on

Status: NEW

(Core :: WebRTC: Networking, defect, P2)

Description

[Nils Ohlmeier [:drno]]

Bug 891551 landed ICE TCP with the user pref set to off by default.

This bug is a reminder that we want/need to switch that pref to on at some point in the future.

Comment 1

[Nils Ohlmeier [:drno]]

For the reference: the pref is called media.peerconnection.ice.tcp

Comment 2

[Nils Ohlmeier [:drno]]

We probably want at least bug 1176407 to be resolved before considering switching it on by default.

Comment 3

[Maire Reavy [:mreavy]]

I know Nils is planning to own and deliver this in Q3 (targeting fx42).

Comment 4

[Nils Ohlmeier [:drno]]

@mreavy: with bug 1266667 landed I think we have the minimum in place to flip the ICE TCP user pref on, which will only activate active ICE TCP connections on e10s and non-e10s.

To reasons to still hold off on flipping the switch are:
1) since bug 1285318 landed we longer have a STUN filter for ICE TCP under e10s.
2) we don't have any good Telemetry data which tells us how often ICE TCP gets used.

Comment 5

[Nils Ohlmeier [:drno]]

(In reply to Nils Ohlmeier [:drno] from comment #4)
> 1) since bug 1285318 landed we longer have a STUN filter for ICE TCP under
> e10s.

Further discussion regarding the missing filter should happen in bug 1285330

Comment 6

[Eric Shepherd [:sheppy]]

I'm told this is preffed on in Firefox 54; I have noted it on Firefox 54 for developers (https://developer.mozilla.org/en-US/Firefox/Releases/54#AudioVideo).

I also added to Experimental features in Firefox (https://developer.mozilla.org/en-US/Firefox/Experimental_features) that this was first implemented in Firefox 41, preffed off.

When I get a link to the blog post I'm told :drno is writing, I'll link to it (unless someone else does first).

That aside, this is now doc complete.

Comment 7

[Chris Mills [:cmills]]

(In reply to Eric Shepherd [:sheppy] from comment #6)
> I'm told this is preffed on in Firefox 54; I have noted it on Firefox 54 for
> developers
> (https://developer.mozilla.org/en-US/Firefox/Releases/54#AudioVideo).
> 
> I also added to Experimental features in Firefox
> (https://developer.mozilla.org/en-US/Firefox/Experimental_features) that
> this was first implemented in Firefox 41, preffed off.
> 
> When I get a link to the blog post I'm told :drno is writing, I'll link to
> it (unless someone else does first).
> 
> That aside, this is now doc complete.

If this is preffed on in Nightly only, I think maybe we should clarify that in the release note? Is that the case here?

Comment 8

[Nils Ohlmeier [:drno]]

(In reply to Chris Mills (Mozilla, MDN editor) [:cmills] from comment #7)
> If this is preffed on in Nightly only, I think maybe we should clarify that
> in the release note? Is that the case here?

No. It is preffed on in 54 and the pref change rides the trains to release (at least that is the plan).

Comment 9

[Bulk Bug Changes for mreavy's org]

Mass change P1->P2 to align with new Mozilla triage process

Comment 10

[Nils Ohlmeier [:drno]]

I don't have the capacity any more to work on this any time soon.