Bug 1301286 - At least in the webrtc49 update, 100Kbps isn't enough for simulcast tests. r=abr, a=test-only
authorRandell Jesup <rjesup@jesup.org>
Thu, 16 Feb 2017 21:29:28 -0500
changeset 376265 afb767a9810a31772c855dd84571f2c598e4a288
parent 376264 2b415e6bea8ef62c98009ace3905130282462230
child 376266 d63931f550626341049889b560f34461143a26ba
push id6996
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 20:48:21 +0000
treeherdermozilla-beta@d89512dab048 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersabr, test-only
bugs1301286
milestone53.0a2
Bug 1301286 - At least in the webrtc49 update, 100Kbps isn't enough for simulcast tests. r=abr, a=test-only MozReview-Commit-ID: kQHNnr7rAg
dom/media/tests/mochitest/test_peerConnection_simulcastOffer.html
--- a/dom/media/tests/mochitest/test_peerConnection_simulcastOffer.html
+++ b/dom/media/tests/mochitest/test_peerConnection_simulcastOffer.html
@@ -21,17 +21,20 @@
     is(receivers.length, 1, "We have exactly one RTP receiver");
     var receiver = receivers[0];
 
     SpecialPowers.wrap(pc._pc).mozSelectSsrc(receiver, index);
   }
 
   runNetworkTest(() =>
     pushPrefs(['media.peerconnection.simulcast', true],
-              ['media.peerconnection.video.min_bitrate_estimate', 100*1000]).then(() => {
+              // 180Kbps was determined empirically, set well-higher than
+              // the 80Kbps+overhead needed for the two simulcast streams.
+              // 100Kbps was apparently too low.
+              ['media.peerconnection.video.min_bitrate_estimate', 180*1000]).then(() => {
       SimpleTest.requestCompleteLog();
       var helper;
 
       test = new PeerConnectionTest({bundle: false});
       test.setMediaConstraints([{video: true}], []);
 
       test.chain.replace("PC_LOCAL_GUM", [
         function PC_LOCAL_CANVAS_CAPTURESTREAM(test) {