Bug 1315318 - Fix reversed info lines in sdpUtils.js verifySdp. r=bwc
authorMichael Froman <mfroman@mozilla.com>
Fri, 04 Nov 2016 12:36:10 -0500
changeset 321117 614eec2873f302035e263746571a6b031057ceec
parent 321116 8ea2413ea36e0a0e250191c77fb3fae05a940356
child 321118 379d17089cc3c5e6dd774e42323988d895804c40
push id30919
push userphilringnalda@gmail.com
push dateSat, 05 Nov 2016 20:28:20 +0000
treeherdermozilla-central@572249b2ffb6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbwc
bugs1315318
milestone52.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1315318 - Fix reversed info lines in sdpUtils.js verifySdp. r=bwc MozReview-Commit-ID: 7DBNaIkyTii
dom/media/tests/mochitest/sdpUtils.js
--- a/dom/media/tests/mochitest/sdpUtils.js
+++ b/dom/media/tests/mochitest/sdpUtils.js
@@ -84,19 +84,19 @@ verifySdp: function(desc, expectedType, 
   ok(desc.sdp.length > 10, "SessionDescription body length is plausible");
   ok(desc.sdp.includes("a=ice-ufrag"), "ICE username is present in SDP");
   ok(desc.sdp.includes("a=ice-pwd"), "ICE password is present in SDP");
   ok(desc.sdp.includes("a=fingerprint"), "ICE fingerprint is present in SDP");
   //TODO: update this for loopback support bug 1027350
   ok(!desc.sdp.includes(LOOPBACK_ADDR), "loopback interface is absent from SDP");
   var requiresTrickleIce = !desc.sdp.includes("a=candidate");
   if (requiresTrickleIce) {
-    info("at least one ICE candidate is present in SDP");
+    info("No ICE candidate in SDP -> requiring trickle ICE");
   } else {
-    info("No ICE candidate in SDP -> requiring trickle ICE");
+    info("at least one ICE candidate is present in SDP");
   }
 
   //TODO: how can we check for absence/presence of m=application?
 
   var audioTracks =
       sdputils.countTracksInConstraint('audio', offerConstraintsList) ||
       ((offerOptions && offerOptions.offerToReceiveAudio) ? 1 : 0);