commit 70bc753cc6c6e80c08f75e63f7a5fb13ea184058
author Åsa Persson <asapersson@webrtc.org> Tue Sep 24 14:28:47 2019
committer Commit Bot <commit-bot@chromium.org> Wed Sep 25 08:15:20 2019
tree ada983bdd54c3e15c1ced3e7efa12c8ef57cb667
parent 84004c441c75b8a7fdf508abdc14832b7d715b19

Add comments to MultiCodecReceiveTest.

Follow up to https://webrtc-review.googlesource.com/c/src/+/153880

Bug: none
Change-Id: If52e2ba638cc463f55330d5d5db1e1e566231562
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/154349
Reviewed-by: Yves Gerey <yvesg@google.com>
Commit-Queue: Åsa Persson <asapersson@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#29294}

video/end_to_end_tests/multi_codec_receive_tests.cc

diff --git a/video/end_to_end_tests/multi_codec_receive_tests.cc b/video/end_to_end_tests/multi_codec_receive_tests.cc
index 44dad57..c12b393 100644
--- a/video/end_to_end_tests/multi_codec_receive_tests.cc
+++ b/video/end_to_end_tests/multi_codec_receive_tests.cc
@@ -115,7 +115,7 @@
   }
 
   rtc::CriticalSection crit_;
-  absl::optional<uint32_t> last_timestamp_;
+  absl::optional<uint32_t> last_timestamp_;  // Only accessed from pacer thread.
   absl::optional<uint8_t> expected_payload_type_ RTC_GUARDED_BY(crit_);
   int num_sent_frames_ RTC_GUARDED_BY(crit_) = 0;
   int num_rendered_frames_ RTC_GUARDED_BY(crit_) = 0;
@@ -212,6 +212,7 @@
     ConfigureEncoder(configs[0]);
     CreateMatchingReceiveConfigs(receive_transport_.get());
     video_receive_configs_[0].renderer = &observer_;
+    // Disable to avoid post-decode frame dropping in VideoRenderFrames.
     video_receive_configs_[0].enable_prerenderer_smoothing = false;
     ConfigureDecoders(configs);
     CreateVideoStreams();