Bug 911046 - Part 6: Make H.264 preferred video codec when enabled in preferences. r=jesup, ekr

media/webrtc/signaling/src/peerconnection/PeerConnectionCtx.cpp

@@ -273,7 +273,12 @@ nsresult PeerConnectionCtx::Initialize() {
   // Only adding codecs supported
   //codecMask |= VCM_CODEC_RESOURCE_H263;
 
-  //codecMask |= VCM_CODEC_RESOURCE_H264;
+#ifdef MOZILLA_INTERNAL_API
+  if (Preferences::GetBool("media.peerconnection.video.h264_enabled")) {
+    codecMask |= VCM_CODEC_RESOURCE_H264;
+  }
+#endif
+
   codecMask |= VCM_CODEC_RESOURCE_VP8;
   //codecMask |= VCM_CODEC_RESOURCE_I420;
   mCCM->setVideoCodecs(codecMask);

media/webrtc/signaling/src/sipcc/core/common/prot_configmgr.c

@@ -614,10 +614,6 @@ sip_config_video_supported_codecs_get (rtp_ptype aSupportedCodecs[],
         //codec_mask = vcmGetVideoCodecList(DSP_ENCODEONLY);
         codec_mask = vcmGetVideoCodecList(VCM_DSP_IGNORE);
     }
-    if ( codec_mask & VCM_CODEC_RESOURCE_VP8) {
-      aSupportedCodecs[count] = RTP_VP8;
-      count++;
-    }
     if ( codec_mask & VCM_CODEC_RESOURCE_H264) {
       /*
        * include payload type for packetization mode 1 only if ucm sis version
@@ -635,6 +631,10 @@ sip_config_video_supported_codecs_get (rtp_ptype aSupportedCodecs[],
       aSupportedCodecs[count] = RTP_H264_P0;
       count++;
     }
+    if ( codec_mask & VCM_CODEC_RESOURCE_VP8) {
+      aSupportedCodecs[count] = RTP_VP8;
+      count++;
+    }
     if ( codec_mask & VCM_CODEC_RESOURCE_H263) {
       aSupportedCodecs[count] = RTP_H263;
       count++;

media/webrtc/signaling/src/sipcc/core/gsm/gsm_sdp.c

@@ -4666,6 +4666,7 @@ gsmsdp_negotiate_rtcp_fb (cc_sdp_t *cc_sdp_p,
          */
         switch (codec) {
             case RTP_VP8:
+            case RTP_I420:
                 fb_types &=
                   sdp_rtcp_fb_nack_to_bitmap(SDP_RTCP_FB_NACK_BASIC) |
                   sdp_rtcp_fb_nack_to_bitmap(SDP_RTCP_FB_NACK_PLI) |