add 'LIBYUV_DISABLE_X86' to msan for unittests

libyuv_test.gyp

Index: libyuv_test.gyp
diff --git a/libyuv_test.gyp b/libyuv_test.gyp
index 0037106679e1b093c3efe27a0544d8db0e9ee248..d60550b4811108b7eb5a59d71953afc73a055446 100644
--- a/libyuv_test.gyp
+++ b/libyuv_test.gyp
@@ -27,6 +27,15 @@
       'export_dependent_settings': [
         '<(DEPTH)/testing/gtest.gyp:gtest',
       ],
+      'conditions': [
+        # MemorySanitizer does not support assembly code yet.
+        # http://crbug.com/344505
+        [ 'msan == 1', {
+          'defines': [
+            'LIBYUV_DISABLE_X86',
+          ],
+        }],
+      ], #conditions

[harryjin, 2016/02/09 19:11:44]

I suppose you put it here to make it clear what the bracket is paired with. But
this is unconventional in a build file.

[fbarchard1, 2016/02/09 19:14:35]

The comment #conditions?  yes its for clarity.  can be hard to see the brackets
paired when there are many conditions.

       'defines': [
         # Enable the following 3 macros to turn off assembly for specified CPU.
         # 'LIBYUV_DISABLE_X86',