Enable transport-cc extension for nightly and early beta
Categories
(Core :: WebRTC: Networking, task, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox77 | --- | fixed |
People
(Reporter: jryans, Assigned: dminor)
References
(Blocks 1 open bug)
Details
(Whiteboard: [jitsi-meet])
Attachments
(2 files)
Bug 1606823 will add support for the transport-cc bandwidth estimation extension behind a pref. This bug will track enabling that by default.
Reporter | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment 1•4 years ago
|
||
If I am reading the patch for bug 1606823, the preference is: media.navigator.video.use_transport_cc and it currently defaults to false.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
Larger try job: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9fe851450f700053465fc7817de9c4b6392ffa50. Smaller try job with the mochitest expectations fixed: https://treeherder.mozilla.org/#/jobs?repo=try&revision=86fb4e0ba82c2217610bd15e8eb7b2a4d327f37f.
Assignee | ||
Comment 3•4 years ago
|
||
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D73354
Updated•4 years ago
|
Pushed by nohlmeier@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/48e305833f6a Enable transport-cc extension for nightly and early beta; r=drno
Comment 6•4 years ago
|
||
Backed out for mda failures on test_peerConnection_basicAudioVideoVerifyExtmap.html
Backout link: https://hg.mozilla.org/integration/autoland/rev/2606cf4f4b47e96d4131fa55596cc21e364955ca
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=300573670&repo=autoland&lineNumber=25553
Pushed by nohlmeier@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bbf40c437947 Enable transport-cc extension for nightly and early beta; r=drno https://hg.mozilla.org/integration/autoland/rev/065c7a87c66d Adjust test expectations for transport-cc; r=drno
Comment 8•4 years ago
|
||
(In reply to Narcis Beleuzu [:NarcisB] from comment #6)
Backed out for mda failures on test_peerConnection_basicAudioVideoVerifyExtmap.html
Backout link: https://hg.mozilla.org/integration/autoland/rev/2606cf4f4b47e96d4131fa55596cc21e364955ca
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=300573670&repo=autoland&lineNumber=25553
Sorry I forgot to land the second patch with the test changes as well. Now the test patch is landed, but I can't reland the pref change :-( Is that something you help with?
Comment 9•4 years ago
|
||
Nils, the pref change (D73354) has landed in https://hg.mozilla.org/integration/autoland/rev/bbf40c437947 . Does it look right to you? Both patches in this bug have landed.
Comment 10•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/bbf40c437947
https://hg.mozilla.org/mozilla-central/rev/065c7a87c66d
Assignee | ||
Updated•4 years ago
|
Comment 11•4 years ago
|
||
Probably the title of this bug should be changed to "Enable transport-cc extension in Nightly" to make that more clear. There will need to be a follow up bug to enable it in release versions if apps like Jitsi want to use it.
Updated•4 years ago
|
Description
•