commit 5d2454437831650917f7ea7abd508c9e3b65be86
author Sergey Silkin <ssilkin@webrtc.org> Tue Jul 09 15:16:18 2024
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jul 10 06:51:47 2024
tree 85e8fc724293296400c6ea628e815da0f6edb61d
parent 9ebf0921ea93d37a7c7709d52e8d249b70ecbd4b

Fix keyframe flag

remove +1 offset.

Bug: webrtc:42225151
Change-Id: Ib735fddfd82f0ae9cfb433648950d936647614a9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/356820
Commit-Queue: Erik Språng <sprang@webrtc.org>
Auto-Submit: Sergey Silkin <ssilkin@webrtc.org>
Reviewed-by: Erik Språng <sprang@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#42616}

modules/video_coding/codecs/test/video_codec_test.cc

diff --git a/modules/video_coding/codecs/test/video_codec_test.cc b/modules/video_coding/codecs/test/video_codec_test.cc
index 3d07a3d..2b144ac 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 @@
   uint32_t timestamp_rtp = 90000;
   std::map<uint32_t, EncodingSettings> 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;
   }