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

Unified Diff: webrtc/pc/mediasession.cc

Issue 2675173003: Adding "adapter" ORTC objects on top of ChannelManager/BaseChannel/etc. (Closed)
Patch Set: Merge with master Created 3 years, 10 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/mediasession.cc
diff --git a/webrtc/pc/mediasession.cc b/webrtc/pc/mediasession.cc
index 000c289c186b9c2daa5f41b0730d96d55f2b2665..e1277ea1cb4aa1f277082c4d29db4960108be882 100644
--- a/webrtc/pc/mediasession.cc
+++ b/webrtc/pc/mediasession.cc
@@ -403,12 +403,10 @@ class UsedPayloadTypes : public UsedIds<Codec> {
class UsedRtpHeaderExtensionIds : public UsedIds<webrtc::RtpExtension> {
public:
UsedRtpHeaderExtensionIds()
- : UsedIds<webrtc::RtpExtension>(kLocalIdMin, kLocalIdMax) {}
+ : UsedIds<webrtc::RtpExtension>(webrtc::RtpExtension::kMinId,
+ webrtc::RtpExtension::kMaxId) {}
private:
- // Min and Max local identifier for one-byte header extensions, per RFC5285.
- static const int kLocalIdMin = 1;
- static const int kLocalIdMax = 14;
};
static bool IsSctp(const MediaContentDescription* desc) {
@@ -1280,7 +1278,6 @@ MediaSessionDescriptionFactory::MediaSessionDescriptionFactory(
transport_desc_factory_(transport_desc_factory) {
channel_manager->GetSupportedAudioSendCodecs(&audio_send_codecs_);
channel_manager->GetSupportedAudioReceiveCodecs(&audio_recv_codecs_);
- channel_manager->GetSupportedAudioSendCodecs(&audio_send_codecs_);
channel_manager->GetSupportedAudioRtpHeaderExtensions(&audio_rtp_extensions_);
channel_manager->GetSupportedVideoCodecs(&video_codecs_);
channel_manager->GetSupportedVideoRtpHeaderExtensions(&video_rtp_extensions_);

Powered by Google App Engine
This is Rietveld 408576698