commit 971f8de35a13c27b967151c28ae4e737a773a26a
author Philipp Hancke <phancke@microsoft.com> Wed Oct 25 12:31:03 2023
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Oct 27 12:38:36 2023
tree 3a42001c98b7818bfe609f34961fcb5cdd535231
parent 9df93c11909c7d22359b266b93c4ab21643ba76f

Remove MediaContentDescriptionImpl<Codec>

after dependencies adopted the RtpMediaContentDescription which
this is currently aliased to.

Also move definition of AudioCodecs and VideoCodecs to the place
where codecs are defined.

BUG=webrtc:15214

Change-Id: I9b0456e1c69c8b23e0cc7665a59baae268872d9c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/325021
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Commit-Queue: Philipp Hancke <phancke@microsoft.com>
Reviewed-by: Florent Castelli <orphis@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#41020}

media/base/codec.h

diff --git a/media/base/codec.h b/media/base/codec.h
index 036a172..228acad 100644
--- a/media/base/codec.h
+++ b/media/base/codec.h
@@ -28,7 +28,7 @@
 
 namespace cricket {
 
-typedef std::map<std::string, std::string> CodecParameterMap;
+using CodecParameterMap = std::map<std::string, std::string>;
 
 class FeedbackParam {
  public:
@@ -187,6 +187,9 @@
 using VideoCodec = Codec;
 using AudioCodec = Codec;
 
+using VideoCodecs = std::vector<Codec>;
+using AudioCodecs = std::vector<Codec>;
+
 Codec CreateAudioCodec(int id,
                        const std::string& name,
                        int clockrate,

pc/session_description.h

diff --git a/pc/session_description.h b/pc/session_description.h
index 044fccf..403e465 100644
--- a/pc/session_description.h
+++ b/pc/session_description.h
@@ -43,10 +43,8 @@
 
 namespace cricket {
 
-typedef std::vector<AudioCodec> AudioCodecs;
-typedef std::vector<VideoCodec> VideoCodecs;
-typedef std::vector<CryptoParams> CryptoParamsVec;
-typedef std::vector<webrtc::RtpExtension> RtpHeaderExtensions;
+using CryptoParamsVec = std::vector<CryptoParams>;
+using RtpHeaderExtensions = std::vector<webrtc::RtpExtension>;
 
 // Options to control how session descriptions are generated.
 const int kAutoBandwidth = -1;
@@ -258,8 +256,9 @@
   }
 
  protected:
-  // TODO(bugs.webrtc.org/15214): move all RTP related things to a subclass that
-  // the SCTP content description does not inherit from.
+  // TODO(bugs.webrtc.org/15214): move all RTP related things to
+  // RtpMediaDescription that the SCTP content description does
+  // not inherit from.
   std::string protocol_;
 
  private:
@@ -290,9 +289,7 @@
   std::vector<Codec> codecs_;
 };
 
-template <class C>
-class MediaContentDescriptionImpl : public MediaContentDescription {};
-using RtpMediaContentDescription = MediaContentDescriptionImpl<Codec>;
+class RtpMediaContentDescription : public MediaContentDescription {};
 
 class AudioContentDescription : public RtpMediaContentDescription {
  public: