Negotiate H264 profiles in SDP

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 14564071da8b42e054319734ffe93cf068f1c9ac..3b91f1e3912d5f29dd93237479619450b56cfc55 100644
--- a/webrtc/media/engine/webrtcvideoengine2.cc
+++ b/webrtc/media/engine/webrtcvideoengine2.cc
@@ -403,6 +403,11 @@ void AddCodecAndMaybeRtxCodec(const VideoCodec& codec,
   } else if (CodecNamesEq(codec.name, kVp9CodecName)) {
     rtx_payload_type = kDefaultRtxVp9PlType;
   } else if (CodecNamesEq(codec.name, kH264CodecName)) {
+    // TODO(magjed): Using a hardcoded payload type for RTX H264 does not work
+    // when we have more than one profile. For example, when having both
+    // Constrained Baseline Profile and Constrained High Profile, both
+    // corresponding RTX codecs will end up with the same payload type, causing
+    // failure later on.
     rtx_payload_type = kDefaultRtxH264PlType;
   } else if (CodecNamesEq(codec.name, kRedCodecName)) {
     rtx_payload_type = kDefaultRtxRedPlType;