commit 084f1ae4bbc73de3880aaba3753adee0fbf3c6c2
author Mirko Bonadei <mbonadei@webrtc.org> Tue Oct 01 06:56:58 2024
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Oct 01 07:36:44 2024
tree aa6a2aaff68aa9d3f6e384446cedf5cf533367fa
parent 865285587267a7bdbf0a9e680b9544154b37e306

Revert "Disable LibaomAv1Encoder tests to unblock Chromium roll"

This reverts commit f8b3dab7c6320a9890f0b003b43d7099e2e00a5b.

Reason for revert: The fix landed in libaom (https://aomedia-review.googlesource.com/c/aom/+/193761) and it is now available in WebRTC (import CL: https://webrtc-review.googlesource.com/c/src/+/364126).

Original change's description:
> Disable LibaomAv1Encoder tests to unblock Chromium roll
>
> The tests exercise the new encoder API that is not used in prod yet.
>
> Bug: webrtc:369633254
> Change-Id: Iee6bc16ebd471f4accdd9531cdb404f159557f51
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/363820
> Commit-Queue: Sergey Silkin <ssilkin@webrtc.org>
> Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
> Cr-Commit-Position: refs/heads/main@{#43083}

Bug: webrtc:369633254
Change-Id: Ia02db32f7f09e3abc3d0a46605feeabd82673f06
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/364281
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Sergey Silkin <ssilkin@webrtc.org>
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Bot-Commit: rubber-stamper@appspot.gserviceaccount.com <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#43120}

api/video_codecs/libaom_av1_encoder_factory_test.cc

diff --git a/api/video_codecs/libaom_av1_encoder_factory_test.cc b/api/video_codecs/libaom_av1_encoder_factory_test.cc
index 594f9fc..0d87a8c 100644
--- a/api/video_codecs/libaom_av1_encoder_factory_test.cc
+++ b/api/video_codecs/libaom_av1_encoder_factory_test.cc
@@ -420,8 +420,8 @@
   EXPECT_THAT(Resolution(f2), ResolutionIs(160, 90));
   EXPECT_THAT(Psnr(in2, f2), Gt(40));
 }
-// TODO(webrtc:369633254): Re-enable after the libaom issue is fixed.
-TEST(LibaomAv1Encoder, DISABLED_TempoSpatial) {
+
+TEST(LibaomAv1Encoder, TempoSpatial) {
   auto frame_reader = CreateFrameReader();
   auto enc = LibaomAv1EncoderFactory().CreateEncoder(kCbrEncoderSettings, {});
 
@@ -507,8 +507,7 @@
   EXPECT_THAT(Resolution(dec.Decode(tu2_s1.bitstream)), ResolutionIs(640, 360));
 }
 
-// TODO(webrtc:369633254): Re-enable after the libaom issue is fixed.
-TEST(LibaomAv1Encoder, DISABLED_SkipMidLayer) {
+TEST(LibaomAv1Encoder, SkipMidLayer) {
   auto frame_reader = CreateFrameReader();
   auto enc = LibaomAv1EncoderFactory().CreateEncoder(kCbrEncoderSettings, {});