Use Xvfb in isolated testing only if use_x11 is set

ash/ash_unittests.isolate

Index: ash/ash_unittests.isolate
diff --git a/ash/ash_unittests.isolate b/ash/ash_unittests.isolate
index 918a8e0252930e5616552d84d938803334d1dfc8..1a6aa163d0d171260a178efc19cc9a03e4d02003 100644
--- a/ash/ash_unittests.isolate
+++ b/ash/ash_unittests.isolate
@@ -15,7 +15,7 @@
         ],
       },
     }],
-    ['OS=="linux"', {
+    ['OS=="linux" and use_x11==1', {

[M-A Ruel, 2015/01/07 21:43:07]

I'd prefer to simplify the condition;

['use_x11==1', {
  'command': [
    xvfb.py PRODUCT_DIR ...
  ],
  'files': [ xvfb.py and xdisplaycheck ],
}
['use_x11==0', {
  'command': [
    '../testing/test_env.py',
    '<(PRODUCT_DIR)/ash_unittests<(EXECUTABLE_SUFFIX)',
    '--test-launcher-bot-mode',
  ],
}

[spang, 2015/01/07 21:59:17]

Done.

       'variables': {
         'command': [
           '../testing/xvfb.py',
@@ -25,29 +25,33 @@
         ],
         'files': [
           '../testing/xvfb.py',
+          '<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
+        ],
+      },
+    }],
+    ['OS=="linux"', {
+      'variables': {
+        'files': [
           '<(PRODUCT_DIR)/libffmpegsumo.so',
           '<(PRODUCT_DIR)/libosmesa.so',
         ],
       },
     }],
-    ['OS=="linux" and use_ozone==0', {
+    ['OS=="win"', {
       'variables': {
         'files': [
-          '<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
+          '<(PRODUCT_DIR)/ffmpegsumo.dll',
+          '<(PRODUCT_DIR)/osmesa.dll',
         ],
       },
     }],
-    ['OS=="win"', {
+    ['OS=="win" or use_ozone==1', {
       'variables': {
         'command': [
           '../testing/test_env.py',
-          '<(PRODUCT_DIR)/ash_unittests.exe',
+          '<(PRODUCT_DIR)/ash_unittests<(EXECUTABLE_SUFFIX)',
           '--test-launcher-bot-mode',
         ],
-        'files': [
-          '<(PRODUCT_DIR)/ffmpegsumo.dll',
-          '<(PRODUCT_DIR)/osmesa.dll',
-        ],
       },
     }],
     ['OS=="win" and (fastbuild==0 or fastbuild==1)', {