diff --git a/api/rtp_sender_interface.cc b/api/rtp_sender_interface.cc index 2b27b7489c..d23fd1844c 100644 --- a/api/rtp_sender_interface.cc +++ b/api/rtp_sender_interface.cc @@ -30,16 +30,4 @@ rtc::scoped_refptr RtpSenderInterface::dtls_transport() return nullptr; } -// TODO(amithi): Fix downstream dependencies and make GetParameters pure -// virtual. -RtpParameters RtpSenderInterface::GetParameters() { - const RtpSenderInterface* interface = this; - return interface->GetParameters(); -} - -RtpParameters RtpSenderInterface::GetParameters() const { - RtpSenderInterface* interface = const_cast(this); - return interface->GetParameters(); -} - } // namespace webrtc diff --git a/api/rtp_sender_interface.h b/api/rtp_sender_interface.h index f8a97575fb..e0a29a33cb 100644 --- a/api/rtp_sender_interface.h +++ b/api/rtp_sender_interface.h @@ -67,9 +67,7 @@ class RtpSenderInterface : public rtc::RefCountInterface { // TODO(orphis): Make it pure virtual once Chrome has updated virtual std::vector init_send_encodings() const; - // TODO(amithi): Fix downstream dependecies and remove the non-const method. - virtual RtpParameters GetParameters(); - virtual RtpParameters GetParameters() const; + virtual RtpParameters GetParameters() const = 0; // Note that only a subset of the parameters can currently be changed. See // rtpparameters.h // The encodings are in increasing quality order for simulcast.