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

Unified Diff: webrtc/pc/mediasession.h

Issue 1984983002: Remove use of RtpHeaderExtension and clean up (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Addressed nit Created 4 years, 7 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/pc/channel.cc ('k') | webrtc/pc/mediasession.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/pc/mediasession.h
diff --git a/webrtc/pc/mediasession.h b/webrtc/pc/mediasession.h
index 22291c42a6412b1f35040489ec26806b3377e010..0a04bcf93f575587d2a82184400e574be41c9e80 100644
--- a/webrtc/pc/mediasession.h
+++ b/webrtc/pc/mediasession.h
@@ -35,7 +35,7 @@ typedef std::vector<AudioCodec> AudioCodecs;
typedef std::vector<VideoCodec> VideoCodecs;
typedef std::vector<DataCodec> DataCodecs;
typedef std::vector<CryptoParams> CryptoParamsVec;
-typedef std::vector<RtpHeaderExtension> RtpHeaderExtensions;
+typedef std::vector<webrtc::RtpExtension> RtpHeaderExtensions;
the sun 2016/05/30 16:00:17 Is this actually used anywhere? ...
enum MediaType {
MEDIA_TYPE_AUDIO,
@@ -205,7 +205,7 @@ class MediaContentDescription : public ContentDescription {
rtp_header_extensions_ = extensions;
rtp_header_extensions_set_ = true;
}
- void AddRtpHeaderExtension(const RtpHeaderExtension& ext) {
+ void AddRtpHeaderExtension(const webrtc::RtpExtension& ext) {
rtp_header_extensions_.push_back(ext);
rtp_header_extensions_set_ = true;
}
@@ -284,7 +284,7 @@ class MediaContentDescription : public ContentDescription {
std::string protocol_;
std::vector<CryptoParams> cryptos_;
CryptoType crypto_required_ = CT_NONE;
- std::vector<RtpHeaderExtension> rtp_header_extensions_;
+ std::vector<webrtc::RtpExtension> rtp_header_extensions_;
the sun 2016/05/30 16:00:17 ...it could be used here.
bool rtp_header_extensions_set_ = false;
bool multistream_ = false;
StreamParamsVec streams_;
« no previous file with comments | « webrtc/pc/channel.cc ('k') | webrtc/pc/mediasession.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698