commit da6a9d58f8770600698ff140f0299dcbf737edd9
author Mirko Bonadei <mbonadei@webrtc.org> Thu Aug 12 09:47:38 2021
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Aug 12 15:24:13 2021
tree 9975562844c73a4c6afdbb0ae2fdcf96a31a3b1f
parent aa918e0111389c12bc564657ba04ff4f42be8031

Use GTEST_SKIP() instead of early return.

The issue for iOS platforms was fixed by crrev.com/c/2505276.

Bug: webrtc:13057
Change-Id: I0f01c1291184f4c7960db746951255961a962303
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/228520
Reviewed-by: Niels Moller <nisse@webrtc.org>
Reviewed-by: Alessio Bazzica <alessiob@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#34740}

system_wrappers/source/denormal_disabler_unittest.cc

diff --git a/system_wrappers/source/denormal_disabler_unittest.cc b/system_wrappers/source/denormal_disabler_unittest.cc
index 32516db..1006c67 100644
--- a/system_wrappers/source/denormal_disabler_unittest.cc
+++ b/system_wrappers/source/denormal_disabler_unittest.cc
@@ -132,8 +132,8 @@
 // compiler are not supported.
 TEST(DenormalDisabler, DoNotZeroDenormalsIfUnsupported) {
   if (DenormalDisabler::IsSupported()) {
-    // TODO(bugs.webrtc.org/13057): Use GTEST_SKIP() once fixed.
-    return;
+    GTEST_SKIP() << "This test should only run on platforms without support "
+                    "for DenormalDisabler.";
   }
   DenormalDisabler denormal_disabler(/*enabled=*/true);
   for (float x : kDenormalDivisors) {