diff --git a/modules/video_coding/svc/scalable_video_controller.h b/modules/video_coding/svc/scalable_video_controller.h index 578346e00c..d2d8486863 100644 --- a/modules/video_coding/svc/scalable_video_controller.h +++ b/modules/video_coding/svc/scalable_video_controller.h @@ -84,9 +84,7 @@ class ScalableVideoController { // Notifies Controller with updated bitrates per layer. In particular notifies // when certain layers should be disabled. // Controller shouldn't produce LayerFrameConfig for disabled layers. - // TODO(bugs.webrtc.org/11404): Make pure virtual when implemented by all - // structures. - virtual void OnRatesUpdated(const VideoBitrateAllocation& bitrates) {} + virtual void OnRatesUpdated(const VideoBitrateAllocation& bitrates) = 0; // When `restart` is true, first `LayerFrameConfig` should have `is_keyframe` // set to true. diff --git a/modules/video_coding/svc/scalable_video_controller_no_layering.h b/modules/video_coding/svc/scalable_video_controller_no_layering.h index 576761d044..e253ffe841 100644 --- a/modules/video_coding/svc/scalable_video_controller_no_layering.h +++ b/modules/video_coding/svc/scalable_video_controller_no_layering.h @@ -13,6 +13,7 @@ #include #include "api/transport/rtp/dependency_descriptor.h" +#include "api/video/video_bitrate_allocation.h" #include "common_video/generic_frame_descriptor/generic_frame_info.h" #include "modules/video_coding/svc/scalable_video_controller.h" @@ -27,6 +28,7 @@ class ScalableVideoControllerNoLayering : public ScalableVideoController { std::vector NextFrameConfig(bool restart) override; GenericFrameInfo OnEncodeDone(const LayerFrameConfig& config) override; + void OnRatesUpdated(const VideoBitrateAllocation& bitrates) override {} private: bool start_ = true;