From 7a841ce116864a4481644656caddf31a7262cfd7 Mon Sep 17 00:00:00 2001 From: Ilya Nikolaevskiy Date: Fri, 24 Nov 2023 13:01:21 +0100 Subject: [PATCH] Fix parameter types inconsistency in FrameCadanceAdapter Bug: chromium:1255737 Change-Id: Id448a9176020f7f88eb767036b67ca884bdbd472 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/328520 Reviewed-by: Markus Handell Reviewed-by: Henrik Andreassson Commit-Queue: Ilya Nikolaevskiy Cr-Commit-Position: refs/heads/main@{#41232} --- video/frame_cadence_adapter.cc | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/video/frame_cadence_adapter.cc b/video/frame_cadence_adapter.cc index 99d4001b72..59e84f9faf 100644 --- a/video/frame_cadence_adapter.cc +++ b/video/frame_cadence_adapter.cc @@ -252,7 +252,7 @@ class FrameCadenceAdapterImpl : public FrameCadenceAdapterInterface { private: // Called from OnFrame in both pass-through and zero-hertz mode. void OnFrameOnMainQueue(Timestamp post_time, - int frames_scheduled_for_processing, + bool queue_overload, const VideoFrame& frame) RTC_RUN_ON(queue_); // Returns true under all of the following conditions: @@ -719,13 +719,11 @@ void FrameCadenceAdapterImpl::OnConstraintsChanged( })); } -void FrameCadenceAdapterImpl::OnFrameOnMainQueue( - Timestamp post_time, - int frames_scheduled_for_processing, - const VideoFrame& frame) { +void FrameCadenceAdapterImpl::OnFrameOnMainQueue(Timestamp post_time, + bool queue_overload, + const VideoFrame& frame) { RTC_DCHECK_RUN_ON(queue_); - current_adapter_mode_->OnFrame(post_time, frames_scheduled_for_processing, - frame); + current_adapter_mode_->OnFrame(post_time, queue_overload, frame); } bool FrameCadenceAdapterImpl::IsZeroHertzScreenshareEnabled() const {