commit c8b857f1c594c06b2124c9e08974d96ae3db3c10
author Sergey Silkin <ssilkin@webrtc.org> Sat Jun 22 11:35:19 2024
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Jun 24 10:59:39 2024
tree 834cfa55bb44772daf58da4fb2a8926fd5eab95a
parent 9603aa1f7e8a304492cd04492476d8fb26fcdb46

Always use SEA in video quality tests

This makes testing closer to production. Chromium always uses SEA: https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/platform/peerconnection/video_codec_factory.cc;l=124

Bug: b/348784414
Change-Id: I95ae2056bf05ff32192a6f49af5fa97c104131a1
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/355280
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Commit-Queue: Sergey Silkin <ssilkin@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#42523}

video/video_quality_test.cc

diff --git a/video/video_quality_test.cc b/video/video_quality_test.cc
index ee2a66d..3753ec1 100644
--- a/video/video_quality_test.cc
+++ b/video/video_quality_test.cc
@@ -317,13 +317,11 @@
     const SdpVideoFormat& format,
     VideoAnalyzer* analyzer) {
   std::unique_ptr<VideoEncoder> encoder;
-  if (format.name == "VP8") {
-    encoder = std::make_unique<SimulcastEncoderAdapter>(
-        env, encoder_factory_.get(), nullptr, format);
-  } else if (format.name == "FakeCodec") {
+  if (format.name == "FakeCodec") {
     encoder = FakeVideoEncoderFactory().Create(env, format);
   } else {
-    encoder = encoder_factory_->Create(env, format);
+    encoder = std::make_unique<SimulcastEncoderAdapter>(
+        env, encoder_factory_.get(), nullptr, format);
   }
 
   std::vector<FileWrapper> encoded_frame_dump_files;