commit 2bc91e8c6af8f9a34699e90f11b6ff03ab410b6f
author Evan Shrubsole <eshr@google.com> Fri Jan 17 10:44:24 2020
committer Commit Bot <commit-bot@chromium.org> Wed Jan 22 15:41:38 2020
tree 3af3e3ca7900ae19ccb714e3306e7c196de25b5d
parent 3986fa8c7eb34959a32f4a4c056a82a4f83bf679

Avoid extra EncodedFrame copy in RunPostEncode

All uses of encoded_image are const, except for the copy for running on
the encoder_queue_.

Bug: None
Change-Id: I7fc8cb46f6afb42a2d27961d3d3ff8d9e63fe1b5
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/166442
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Evan Shrubsole <eshr@google.com>
Cr-Commit-Position: refs/heads/master@{#30351}

video/video_stream_encoder.cc

diff --git a/video/video_stream_encoder.cc b/video/video_stream_encoder.cc
index 486143c..68f24e6 100644
--- a/video/video_stream_encoder.cc
+++ b/video/video_stream_encoder.cc
@@ -1761,7 +1761,7 @@
   video_source_sink_controller_->PushSourceSinkSettings();
 }
 
-void VideoStreamEncoder::RunPostEncode(EncodedImage encoded_image,
+void VideoStreamEncoder::RunPostEncode(const EncodedImage& encoded_image,
                                        int64_t time_sent_us,
                                        int temporal_index,
                                        DataSize frame_size) {

video/video_stream_encoder.h

diff --git a/video/video_stream_encoder.h b/video/video_stream_encoder.h
index 5ac6db8..a905420 100644
--- a/video/video_stream_encoder.h
+++ b/video/video_stream_encoder.h
@@ -195,7 +195,7 @@
   void SetEncoderRates(const EncoderRateSettings& rate_settings)
       RTC_RUN_ON(&encoder_queue_);
 
-  void RunPostEncode(EncodedImage encoded_image,
+  void RunPostEncode(const EncodedImage& encoded_image,
                      int64_t time_sent_us,
                      int temporal_index,
                      DataSize frame_size);