From fa67bcc27a450ef336fa7cc98c2770376470ce6c Mon Sep 17 00:00:00 2001
From: Philipp Hancke <phancke@microsoft.com>
Date: Wed, 7 Dec 2022 08:59:24 +0100
Subject: [PATCH] generateKeyFrame: use GetParametersInternal

since the externally visible GetParameters() method modifies
the last transaction id which is neither required nor desirable
for generateKeyFrame.

BUG=chromium:1354101

Change-Id: Iae9ea36f78258d3be6d02eb2858c8bf9c4c672c8
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/286740
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Commit-Queue: Philipp Hancke <phancke@microsoft.com>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#38911}
---
 pc/rtp_sender.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pc/rtp_sender.cc b/pc/rtp_sender.cc
index b3330d3897..1bbca3c41a 100644
--- a/pc/rtp_sender.cc
+++ b/pc/rtp_sender.cc
@@ -795,7 +795,7 @@ RTCError VideoRtpSender::GenerateKeyFrame(
     const std::vector<std::string>& rids) {
   RTC_DCHECK_RUN_ON(signaling_thread_);
   if (video_media_channel() && ssrc_ && !stopped_) {
-    auto parameters = GetParameters();
+    const auto parameters = GetParametersInternal();
     for (const auto& rid : rids) {
       if (rid.empty()) {
         LOG_AND_RETURN_ERROR(RTCErrorType::INVALID_PARAMETER,