From 5d2454437831650917f7ea7abd508c9e3b65be86 Mon Sep 17 00:00:00 2001 From: Sergey Silkin Date: Tue, 9 Jul 2024 17:16:18 +0200 Subject: [PATCH] Fix keyframe flag MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit remove +1 offset. Bug: webrtc:42225151 Change-Id: Ib735fddfd82f0ae9cfb433648950d936647614a9 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/356820 Commit-Queue: Erik Språng Auto-Submit: Sergey Silkin Reviewed-by: Erik Språng Cr-Commit-Position: refs/heads/main@{#42616} --- modules/video_coding/codecs/test/video_codec_test.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/modules/video_coding/codecs/test/video_codec_test.cc b/modules/video_coding/codecs/test/video_codec_test.cc index 3d07a3d1f7..2b144ac5da 100644 --- a/modules/video_coding/codecs/test/video_codec_test.cc +++ b/modules/video_coding/codecs/test/video_codec_test.cc @@ -582,7 +582,8 @@ TEST(VideoCodecTest, DISABLED_EncodeDecode) { uint32_t timestamp_rtp = 90000; std::map frame_settings; for (int frame_num = 0; frame_num < num_frames; ++frame_num) { - encoding_settings.keyframe = (frame_num % (key_interval + 1) == 0); + encoding_settings.keyframe = + (key_interval > 0 && (frame_num % key_interval) == 0); frame_settings.emplace(timestamp_rtp, encoding_settings); timestamp_rtp += k90kHz / framerate; }