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

Unified Diff: webrtc/pc/channel.h

Issue 2761143002: Support encrypted RTP extensions (RFC 6904) (Closed)
Patch Set: Don't negotiate extension ids in SrtpFilter, more changes after feedback from Taylor. 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/pc/channel.h
diff --git a/webrtc/pc/channel.h b/webrtc/pc/channel.h
index 6ff0556c9798525ffbe3d29b739dc55085c8681c..0e672f477c1ea3898cd7de620f92eae3d27fcf53 100644
--- a/webrtc/pc/channel.h
+++ b/webrtc/pc/channel.h
@@ -329,13 +329,18 @@ class BaseChannel
ContentAction action,
std::string* error_desc) = 0;
bool SetRtpTransportParameters(const MediaContentDescription* content,
- ContentAction action,
- ContentSource src,
- std::string* error_desc);
+ ContentAction action, ContentSource src,
+ const RtpHeaderExtensions& extensions, std::string* error_desc);
bool SetRtpTransportParameters_n(const MediaContentDescription* content,
- ContentAction action,
- ContentSource src,
- std::string* error_desc);
+ ContentAction action, ContentSource src,
+ const std::vector<int>& encrypted_extension_ids,
+ std::string* error_desc);
+
+ // Return a list of RTP header extensions with the non-encrypted extensions
+ // removed depending on the current crypto_options_ and only if both the
+ // non-encrypted and encrypted extension is present for the same URI.
+ RtpHeaderExtensions GetFilteredRtpHeaderExtensions(
+ const RtpHeaderExtensions& extensions);
// Helper method to get RTP Absoulute SendTime extension header id if
// present in remote supported extensions list.
@@ -348,6 +353,7 @@ class BaseChannel
bool SetSrtp_n(const std::vector<CryptoParams>& params,
ContentAction action,
ContentSource src,
+ const std::vector<int>& encrypted_extension_ids,
std::string* error_desc);
bool SetRtcpMux_n(bool enable,
ContentAction action,

Powered by Google App Engine
This is Rietveld 408576698