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

Unified Diff: webrtc/pc/channel.h

Issue 2761143002: Support encrypted RTP extensions (RFC 6904) (Closed)
Patch Set: More updates + support for adding/changing encrypted extensions. Created 3 years, 9 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..e13d32ffde4606c7ffbfbb2a807d2dcc51840800 100644
--- a/webrtc/pc/channel.h
+++ b/webrtc/pc/channel.h
@@ -329,13 +329,17 @@ 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 RtpHeaderExtensions& encrypted_extensions, 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 +352,7 @@ class BaseChannel
bool SetSrtp_n(const std::vector<CryptoParams>& params,
ContentAction action,
ContentSource src,
+ const RtpHeaderExtensions& encrypted_extensions,
std::string* error_desc);
bool SetRtcpMux_n(bool enable,
ContentAction action,
@@ -421,6 +426,7 @@ class BaseChannel
bool has_received_packet_ = false;
bool dtls_keyed_ = false;
const bool srtp_required_ = true;
+ bool srtp_update_on_answer_ = false;
rtc::CryptoOptions crypto_options_;
int rtp_abs_sendtime_extn_id_ = -1;

Powered by Google App Engine
This is Rietveld 408576698