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

Unified Diff: webrtc/media/engine/payload_type_mapper.cc

Issue 2513633002: Revert of Stop using hardcoded payload types for video codecs (Closed)
Patch Set: Created 4 years, 1 month 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/media/engine/internalencoderfactory.cc ('k') | webrtc/media/engine/payload_type_mapper_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/media/engine/payload_type_mapper.cc
diff --git a/webrtc/media/engine/payload_type_mapper.cc b/webrtc/media/engine/payload_type_mapper.cc
index 0d449c4e4320872b2bb2674f492877af980838a1..477c7ab0931572ebeb86b1b9ff49df1e3365272b 100644
--- a/webrtc/media/engine/payload_type_mapper.cc
+++ b/webrtc/media/engine/payload_type_mapper.cc
@@ -53,18 +53,41 @@
{{"G729", 8000, 1}, 18},
// Payload type assignments currently used by WebRTC.
- // Includes data to reduce collisions (and thus reassignments)
+ // Includes video and data to reduce collisions (and thus
+ // reassignments).
// RTX codecs mapping to specific video payload types
+ {{kRtxCodecName, 90000, 0,
+ {{kCodecParamAssociatedPayloadType,
+ std::to_string(kDefaultVp8PlType)}}},
+ kDefaultRtxVp8PlType},
+ {{kRtxCodecName, 90000, 0,
+ {{kCodecParamAssociatedPayloadType,
+ std::to_string(kDefaultVp9PlType)}}},
+ kDefaultRtxVp9PlType},
+ {{kRtxCodecName, 90000, 0,
+ {{kCodecParamAssociatedPayloadType,
+ std::to_string(kDefaultRedPlType)}}},
+ kDefaultRtxRedPlType},
+ {{kRtxCodecName, 90000, 0,
+ {{kCodecParamAssociatedPayloadType,
+ std::to_string(kDefaultH264PlType)}}},
+ kDefaultRtxH264ConstrainedBaselinePlType},
// Other codecs
+ {{kVp8CodecName, 90000, 0}, kDefaultVp8PlType},
+ {{kVp9CodecName, 90000, 0}, kDefaultVp9PlType},
{{kGoogleRtpDataCodecName, 0, 0}, kGoogleRtpDataCodecPlType},
{{kIlbcCodecName, 8000, 1}, 102},
{{kIsacCodecName, 16000, 1}, 103},
{{kIsacCodecName, 32000, 1}, 104},
{{kCnCodecName, 16000, 1}, 105},
{{kCnCodecName, 32000, 1}, 106},
+ {{kH264CodecName, 90000, 0}, kDefaultH264PlType},
{{kGoogleSctpDataCodecName, 0, 0}, kGoogleSctpDataCodecPlType},
{{kOpusCodecName, 48000, 2,
{{"minptime", "10"}, {"useinbandfec", "1"}}}, 111},
+ {{kRedCodecName, 90000, 0}, kDefaultRedPlType},
+ {{kUlpfecCodecName, 90000, 0}, kDefaultUlpfecType},
+ {{kFlexfecCodecName, 90000, 0}, kDefaultFlexfecPlType},
// TODO(solenberg): Remove the hard coded 16k,32k,48k DTMF once we
// assign payload types dynamically for send side as well.
{{kDtmfCodecName, 48000, 1}, 110},
« no previous file with comments | « webrtc/media/engine/internalencoderfactory.cc ('k') | webrtc/media/engine/payload_type_mapper_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698