Lift MEDIA_MMX_INTRINSICS_AVAILABLE definition to header and use it to condition media_unittests

media/base/yuv_convert_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/base_paths.h"
 #include "base/cpu.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/path_service.h"
 #include "media/base/djb2.h"
@@ skipping 757 matching lines @@
   memset(dst.get(), 0, 16);
 
   media::FilterYUVRows_C(dst.get(), src.get(), src.get(), 1, 255);
 
   EXPECT_EQ(255u, dst[0]);
   for (int i = 1; i < 16; ++i) {
     EXPECT_EQ(0u, dst[i]) << " not equal at " << i;
   }
 }
 
 TEST(YUVConvertTest, FilterYUVRows_MMX_OutOfBounds) {

[scherkus (not reviewing), 2013/02/06 08:09:01]

gtest supports disabling tests by prepending the test name with DISABLED_

this way the code is still compiled + linked, but is skipped by default

can you change to the following?

#if defined(MEDIA_MMX_INTRINSICS_AVAILABLE)
TEST(YUVConvertTest, DISABLED_FilterYUVRows_MMX_OutOfBounds) {
#else
TEST(YUVConvertTest, FilterYUVRows_MMX_OutOfBounds) {
#endif

[wolenetz, 2013/02/06 19:03:10]

Sorry about being pedantic here.  Please help me understand your comment:
If I understand correctly, the logic would be reverse of your proposal: if MMX
intrinsics are available, then go ahead and test the method if the cpu also
supports mmx, otherwise disable the test.
On win64, the disabled, but still compiled and linked test, would not enable
successful build because the routine that uses MMX intrinsics wouldn't be built
into media.dll, so linking would fail. On win64, the routine is excluded from
media.dll because it uses MMX intrinsics that VS2010 doesn't support
(compilation of media.dll would fail if included).

If the scope of the win64-disabled version of the test were changed to be
essentially a noop (or the current LOG(WARNING)), then using DISABLED_ would
work, I think.

+#if defined(MEDIA_MMX_INTRINSICS_AVAILABLE)
   base::CPU cpu;
   if (!cpu.has_mmx()) {
     LOG(WARNING) << "System not supported. Test skipped.";
     return;
   }
 
   scoped_array<uint8> src(new uint8[16]);
   scoped_array<uint8> dst(new uint8[16]);
 
   memset(src.get(), 0xff, 16);
   memset(dst.get(), 0, 16);
 
   media::FilterYUVRows_MMX(dst.get(), src.get(), src.get(), 1, 255);
   media::EmptyRegisterState();
 
   EXPECT_EQ(255u, dst[0]);
   for (int i = 1; i < 16; ++i) {
     EXPECT_EQ(0u, dst[i]);
   }
+#else
+  LOG(WARNING) << "System not supported. Test skipped.";
+  return;
+#endif  // defined(MEDIA_MMX_INTRINSICS_AVAILABLE)
 }
 
 TEST(YUVConvertTest, FilterYUVRows_SSE2_OutOfBounds) {
   base::CPU cpu;
   if (!cpu.has_sse2()) {
     LOG(WARNING) << "System not supported. Test skipped.";
     return;
   }
 
   scoped_array<uint8> src(new uint8[16]);
   scoped_array<uint8> dst(new uint8[16]);
 
   memset(src.get(), 0xff, 16);
   memset(dst.get(), 0, 16);
 
   media::FilterYUVRows_SSE2(dst.get(), src.get(), src.get(), 1, 255);
 
   EXPECT_EQ(255u, dst[0]);
   for (int i = 1; i < 16; ++i) {
     EXPECT_EQ(0u, dst[i]);
   }
 }
 
 TEST(YUVConvertTest, FilterYUVRows_MMX_UnalignedDestination) {
+#if defined(MEDIA_MMX_INTRINSICS_AVAILABLE)

[scherkus (not reviewing), 2013/02/06 08:09:01]

ditto

[wolenetz, 2013/02/06 19:03:10]

See reply for similar comment, above.

   base::CPU cpu;
   if (!cpu.has_mmx()) {
     LOG(WARNING) << "System not supported. Test skipped.";
     return;
   }
 
   const int kSize = 32;
   scoped_array<uint8> src(new uint8[kSize]);
   scoped_array<uint8> dst_sample(new uint8[kSize]);
   scoped_array<uint8> dst(new uint8[kSize]);
 
   memset(dst_sample.get(), 0, kSize);
   memset(dst.get(), 0, kSize);
   for (int i = 0; i < kSize; ++i)
     src[i] = 100 + i;
 
   media::FilterYUVRows_C(dst_sample.get(),
                          src.get(), src.get(), 17, 128);
 
   // Generate an unaligned output address.
   uint8* dst_ptr =
       reinterpret_cast<uint8*>(
           (reinterpret_cast<uintptr_t>(dst.get() + 8) & ~7) + 1);
   media::FilterYUVRows_MMX(dst_ptr, src.get(), src.get(), 17, 128);
   media::EmptyRegisterState();
 
   EXPECT_EQ(0, memcmp(dst_sample.get(), dst_ptr, 17));
+#else
+  LOG(WARNING) << "System not supported. Test skipped.";
+  return;
+#endif  // defined(MEDIA_MMX_INTRINSICS_AVAILABLE)
 }
 
 TEST(YUVConvertTest, FilterYUVRows_SSE2_UnalignedDestination) {
   base::CPU cpu;
   if (!cpu.has_sse2()) {
     LOG(WARNING) << "System not supported. Test skipped.";
     return;
   }
 
   const int kSize = 64;
@@ skipping 69 matching lines @@
                                    kSourceDx);
   media::EmptyRegisterState();
   EXPECT_EQ(0, memcmp(rgb_bytes_reference.get(),
                       rgb_bytes_converted.get(),
                       kWidth * kBpp));
 }
 
 #endif  // defined(ARCH_CPU_X86_64)
 
 #endif  // defined(ARCH_CPU_X86_FAMILY)