Chromium Code Reviews| 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 { |