Bug 1242324: Give VP9 priority over vp8 when both are enabled r=drno
authorRandell Jesup <rjesup@jesup.org>
Mon, 25 Jan 2016 01:46:08 -0500
changeset 303689 329f6c0221e8101b9c3868a9bf4461cc4eefd243
parent 303688 9807e39dc5bc53e34d0885a3af68f42f04a0fdf3
child 303690 d5a22b8ad0dd9276837e1c2387daf516433ccb49
push id8978
push userraliiev@mozilla.com
push dateMon, 25 Jan 2016 14:05:32 +0000
treeherdermozilla-aurora@b9a803752a2c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdrno
bugs1242324
milestone46.0a1
Bug 1242324: Give VP9 priority over vp8 when both are enabled r=drno
media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
--- a/media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
+++ b/media/webrtc/signaling/src/jsep/JsepSessionImpl.cpp
@@ -2083,36 +2083,37 @@ JsepSessionImpl::SetupDefaultCodecs()
                                     "PCMA",
                                     8000,
                                     1,
                                     8000 / 50,   // frequency / 50
                                     8 * 8000 * 1 // 8 * frequency * channels
                                     ));
 
   // Supported video codecs.
+  // Note: order here implies priority for building offers!
+  JsepVideoCodecDescription* vp9 = new JsepVideoCodecDescription(
+      "121",
+      "VP9",
+      90000
+      );
+  // Defaults for mandatory params
+  vp9->mConstraints.maxFs = 12288; // Enough for 2048x1536
+  vp9->mConstraints.maxFps = 60;
+  mSupportedCodecs.values.push_back(vp9);
+
   JsepVideoCodecDescription* vp8 = new JsepVideoCodecDescription(
       "120",
       "VP8",
       90000
       );
   // Defaults for mandatory params
   vp8->mConstraints.maxFs = 12288; // Enough for 2048x1536
   vp8->mConstraints.maxFps = 60;
   mSupportedCodecs.values.push_back(vp8);
 
-  JsepVideoCodecDescription* vp9 = new JsepVideoCodecDescription(
-      "121",
-      "VP9",
-      90000
-      );
-  // Defaults for mandatory params
-  vp9->mConstraints.maxFs = 12288; // Enough for 2048x1536
-  vp9->mConstraints.maxFps = 60;
-  mSupportedCodecs.values.push_back(vp9);
-
   JsepVideoCodecDescription* h264_1 = new JsepVideoCodecDescription(
       "126",
       "H264",
       90000
       );
   h264_1->mPacketizationMode = 1;
   // Defaults for mandatory params
   h264_1->mProfileLevelId = 0x42E00D;