Make RtpSenderInternal::CheckCodecParameters pure virtual
Noted that the default implementation wasn't exercised by tests,
so worth removing.
Bug: None
Change-Id: I007ca54724ed27a8c37f34b9eaa188b0d46dd4e8
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/327300
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#41143}
diff --git a/pc/rtp_sender.h b/pc/rtp_sender.h
index 26adceb..8925230 100644
--- a/pc/rtp_sender.h
+++ b/pc/rtp_sender.h
@@ -86,9 +86,7 @@
const RtpParameters& parameters) = 0;
// Additional checks that are specific to the current codec settings
- virtual RTCError CheckCodecParameters(const RtpParameters& parameters) {
- return RTCError::OK();
- }
+ virtual RTCError CheckCodecParameters(const RtpParameters& parameters) = 0;
// Returns an ID that changes every time SetTrack() is called, but
// otherwise remains constant. Used to generate IDs for stats.
