Add read support of RtpStreamId/RepairedRtpStreamId header extensions.

webrtc/modules/rtp_rtcp/source/rtp_utility.cc

Index: webrtc/modules/rtp_rtcp/source/rtp_utility.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_utility.cc b/webrtc/modules/rtp_rtcp/source/rtp_utility.cc
index def431f1709499993bac608a993aa0c49e33bc1b..6cef43f7251784387290b35a61f3c5dd1097e5d8 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_utility.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_utility.cc
@@ -446,6 +446,18 @@ void RtpHeaderParser::ParseOneByteExtensionHeader(
               max_playout_delay * PlayoutDelayLimits::kGranularityMs;
           break;
         }
+        case kRtpExtensionRtpStreamId: {
+          RTC_DCHECK_LE(len + 1, sizeof(header->extension.stream_id));

[nisse-webrtc, 2017/04/10 07:15:27]

len+1 here is a bit confusing, but I guess that's how |len| is used in the rest
of this function?

[danilchap, 2017/04/10 08:39:41]

Agree, I'll make a small follow up cl to make len value in this function more
readable.

+          header->extension.stream_id_size = len + 1;
+          memcpy(header->extension.stream_id, ptr, len + 1);
+          break;
+        }
+        case kRtpExtensionRepairedRtpStreamId: {
+          RTC_DCHECK_LE(len + 1, sizeof(header->extension.repaired_stream_id));
+          header->extension.repaired_stream_id_size = len + 1;
+          memcpy(header->extension.repaired_stream_id, ptr, len + 1);
+          break;
+        }
         case kRtpExtensionNone:
         case kRtpExtensionNumberOfExtensions: {
           RTC_NOTREACHED() << "Invalid extension type: " << type;