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

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

Issue 2761143002: Support encrypted RTP extensions (RFC 6904) (Closed)
Patch Set: Fix compile error on win_x64 bots. Created 3 years, 6 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
« no previous file with comments | « webrtc/media/engine/webrtcmediaengine.cc ('k') | webrtc/p2p/base/dtlstransportchannel.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/media/engine/webrtcmediaengine_unittest.cc
diff --git a/webrtc/media/engine/webrtcmediaengine_unittest.cc b/webrtc/media/engine/webrtcmediaengine_unittest.cc
index 7a97c1a0b9508a383cf228e714c1ef42901374d2..d22a8da7571fc4d2b489a23465dba16a63f74354 100644
--- a/webrtc/media/engine/webrtcmediaengine_unittest.cc
+++ b/webrtc/media/engine/webrtcmediaengine_unittest.cc
@@ -145,6 +145,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(
@@ -160,6 +192,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));
« no previous file with comments | « webrtc/media/engine/webrtcmediaengine.cc ('k') | webrtc/p2p/base/dtlstransportchannel.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698