Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(62)

Unified Diff: webrtc/media/engine/webrtcmediaengine_unittest.cc

Issue 2761143002: Support encrypted RTP extensions (RFC 6904) (Closed)
Patch Set: Updated comment. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/media/engine/webrtcmediaengine_unittest.cc
diff --git a/webrtc/media/engine/webrtcmediaengine_unittest.cc b/webrtc/media/engine/webrtcmediaengine_unittest.cc
index c58c75b0c3a81a15e3089b3b29748565f44758e2..c0b3e16f9de10d79c11a7bb330b6c5835a4c8405 100644
--- a/webrtc/media/engine/webrtcmediaengine_unittest.cc
+++ b/webrtc/media/engine/webrtcmediaengine_unittest.cc
@@ -144,6 +144,38 @@ TEST(WebRtcMediaEngineTest, FilterRtpExtensions_RemoveRedundant) {
EXPECT_NE(filtered[0].uri, filtered[1].uri);
}
+TEST(WebRtcMediaEngineTest, FilterRtpExtensions_RemoveRedundantEncrypted_1) {
+ std::vector<RtpExtension> extensions;
+ extensions.push_back(webrtc::RtpExtension("b", 1));
+ extensions.push_back(webrtc::RtpExtension("b", 2, true));
+ extensions.push_back(webrtc::RtpExtension("c", 3));
+ extensions.push_back(webrtc::RtpExtension("b", 4));
+ std::vector<webrtc::RtpExtension> filtered =
+ FilterRtpExtensions(extensions, SupportedExtensions2, true);
+ EXPECT_EQ(3, filtered.size());
+ EXPECT_TRUE(IsSorted(filtered));
+ EXPECT_EQ(filtered[0].uri, filtered[1].uri);
+ EXPECT_NE(filtered[0].encrypt, filtered[1].encrypt);
+ EXPECT_NE(filtered[0].uri, filtered[2].uri);
+ EXPECT_NE(filtered[1].uri, filtered[2].uri);
+}
+
+TEST(WebRtcMediaEngineTest, FilterRtpExtensions_RemoveRedundantEncrypted_2) {
+ std::vector<RtpExtension> extensions;
+ extensions.push_back(webrtc::RtpExtension("b", 1, true));
+ extensions.push_back(webrtc::RtpExtension("b", 2));
+ extensions.push_back(webrtc::RtpExtension("c", 3));
+ extensions.push_back(webrtc::RtpExtension("b", 4));
+ std::vector<webrtc::RtpExtension> filtered =
+ FilterRtpExtensions(extensions, SupportedExtensions2, true);
+ EXPECT_EQ(3, filtered.size());
+ EXPECT_TRUE(IsSorted(filtered));
+ EXPECT_EQ(filtered[0].uri, filtered[1].uri);
+ EXPECT_NE(filtered[0].encrypt, filtered[1].encrypt);
+ EXPECT_NE(filtered[0].uri, filtered[2].uri);
+ EXPECT_NE(filtered[1].uri, filtered[2].uri);
+}
+
TEST(WebRtcMediaEngineTest, FilterRtpExtensions_RemoveRedundantBwe_1) {
std::vector<RtpExtension> extensions;
extensions.push_back(
@@ -159,6 +191,27 @@ TEST(WebRtcMediaEngineTest, FilterRtpExtensions_RemoveRedundantBwe_1) {
EXPECT_EQ(RtpExtension::kTransportSequenceNumberUri, filtered[0].uri);
}
+TEST(WebRtcMediaEngineTest, FilterRtpExtensions_RemoveRedundantBweEncrypted_1) {
+ std::vector<RtpExtension> extensions;
+ extensions.push_back(
+ RtpExtension(RtpExtension::kTransportSequenceNumberUri, 3));
+ extensions.push_back(
+ RtpExtension(RtpExtension::kTransportSequenceNumberUri, 4, true));
+ extensions.push_back(RtpExtension(RtpExtension::kTimestampOffsetUri, 9));
+ extensions.push_back(RtpExtension(RtpExtension::kAbsSendTimeUri, 6));
+ extensions.push_back(
+ RtpExtension(RtpExtension::kTransportSequenceNumberUri, 1));
+ extensions.push_back(
+ RtpExtension(RtpExtension::kTransportSequenceNumberUri, 2, true));
+ extensions.push_back(RtpExtension(RtpExtension::kTimestampOffsetUri, 14));
+ std::vector<webrtc::RtpExtension> filtered =
+ FilterRtpExtensions(extensions, SupportedExtensions2, true);
+ EXPECT_EQ(2, filtered.size());
+ EXPECT_EQ(RtpExtension::kTransportSequenceNumberUri, filtered[0].uri);
+ EXPECT_EQ(RtpExtension::kTransportSequenceNumberUri, filtered[1].uri);
+ EXPECT_NE(filtered[0].encrypt, filtered[1].encrypt);
+}
+
TEST(WebRtcMediaEngineTest, FilterRtpExtensions_RemoveRedundantBwe_2) {
std::vector<RtpExtension> extensions;
extensions.push_back(RtpExtension(RtpExtension::kTimestampOffsetUri, 1));

Powered by Google App Engine
This is Rietveld 408576698