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

Unified Diff: webrtc/config.h

Issue 2761143002: Support encrypted RTP extensions (RFC 6904) (Closed)
Patch Set: 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/config.h
diff --git a/webrtc/config.h b/webrtc/config.h
index f8c9e8b7971724835a4b62b47b038ed40a61a903..5be84a84bf363ecb13f76f43c736af17d2bcbc10 100644
--- a/webrtc/config.h
+++ b/webrtc/config.h
@@ -58,14 +58,17 @@ struct UlpfecConfig {
// RTP header extension, see RFC 5285.
struct RtpExtension {
- RtpExtension() : id(0) {}
+ RtpExtension() {}
RtpExtension(const std::string& uri, int id) : uri(uri), id(id) {}
+ RtpExtension(const std::string& uri, int id, bool encrypted) : uri(uri),
+ id(id), encrypted(encrypted) {}
std::string ToString() const;
bool operator==(const RtpExtension& rhs) const {
- return uri == rhs.uri && id == rhs.id;
+ return uri == rhs.uri && id == rhs.id && encrypted == rhs.encrypted;
}
static bool IsSupportedForAudio(const std::string& uri);
static bool IsSupportedForVideo(const std::string& uri);
+ static bool AllowEncrypt(const std::string& uri);
Taylor Brandstetter 2017/03/22 18:00:10 Can you leave a comment describing this method?
joachim 2017/03/23 00:04:32 Done.
// Header extension for audio levels, as defined in:
// http://tools.ietf.org/html/draft-ietf-avtext-client-to-mixer-audio-level-03
@@ -96,12 +99,17 @@ struct RtpExtension {
static const char* kPlayoutDelayUri;
static const int kPlayoutDelayDefaultId;
+ // Encryption of Header Extensions, see RFC 6904 for details:
+ // https://tools.ietf.org/html/rfc6904
+ static const char* kEncryptHeaderExtensionsUri;
+
// Inclusive min and max IDs for one-byte header extensions, per RFC5285.
static const int kMinId;
static const int kMaxId;
std::string uri;
- int id;
+ int id = 0;
+ bool encrypted = false;
Taylor Brandstetter 2017/03/22 18:00:10 nit: If this was just "encrypt" rather than "encry
joachim 2017/03/23 00:04:32 Done.
};
struct VideoStream {

Powered by Google App Engine
This is Rietveld 408576698