Generate FMTP parameters for the H.264 codec.

webrtc/media/engine/webrtcvideoengine2.cc

Index: webrtc/media/engine/webrtcvideoengine2.cc
diff --git a/webrtc/media/engine/webrtcvideoengine2.cc b/webrtc/media/engine/webrtcvideoengine2.cc
index ec6b033d2f7a3862be69a9c25390d646968cfd2c..508ab19d5180ac65947d0b48d694ba1abb65bdc1 100644
--- a/webrtc/media/engine/webrtcvideoengine2.cc
+++ b/webrtc/media/engine/webrtcvideoengine2.cc
@@ -386,8 +386,15 @@ std::vector<VideoCodec> DefaultVideoCodecList() {
         VideoCodec::CreateRtxCodec(kDefaultRtxVp9PlType, kDefaultVp9PlType));
   }
   if (CodecIsInternallySupported(kH264CodecName)) {

[hbos, 2016/04/12 11:41:14]

Hmm, I guess this was always the case, but this conditional means the SDP will
only be correctly offered if there are internal H.264 implementations available.
Even if an external factory (Chrome) provides both HW encoder and HW decoder, an
internal implementation must be available - and in that case unused - for the
SDP to contain H264.

I suppose it's correct behavior for the "default codec list" to only consider
"default" (internal) codecs. But how is the case handled for purely external
codecs, so that those codecs are offered in the SDP? If that case isn't handled,
there must be an an implicit assumption somewhere about the default codec list
being complete when offering SDP. Or, if that case is handled, perhaps there is
another place in the code where we should modify the codec list.

- Should we add a TODO before landing?

[hta-webrtc, 2016/04/12 12:48:44]

Yes, that makes the third TODO here: Move SDP parameter generation for codecs to
depend on the codec implementation, not on this file.

We have another effort (somewhere) to remove the idea of "internal codecs"
altogether and only have a "default codec list" that contains the codecs that
are included in the compilation. That will hopefully revisit this piece of code!

-    codecs.push_back(MakeVideoCodecWithDefaultFeedbackParams(kDefaultH264PlType,
-                                                             kH264CodecName));
+    VideoCodec codec = MakeVideoCodecWithDefaultFeedbackParams(
+        kDefaultH264PlType, kH264CodecName);
+    // TODO(hta): Move selection of profile-level-id to codec implementation.
+    // TODO(hta): Set FMTP parameters for all codecs of type H264.
+    codec.SetParam(kH264FmtpProfileLevelId,
+                   kH264ProfileLevelConstrainedBaseline);
+    codec.SetParam(kH264FmtpLevelAsymmetryAllowed, "1");
+    codec.SetParam(kH264FmtpPacketizationMode, "1");
+    codecs.push_back(codec);
     codecs.push_back(
         VideoCodec::CreateRtxCodec(kDefaultRtxH264PlType, kDefaultH264PlType));
   }