From 9589e2af1642fce385fb8c47e3726a5c416a4e02 Mon Sep 17 00:00:00 2001
From: Henrik Kjellander <kjellander@webrtc.org>
Date: Thu, 22 Oct 2015 06:48:21 +0200
Subject: [PATCH] Update isolate files for swarming tests

Xvfb is needed for the screen capture tests in modules_unittests,
which also brings in xdisplaycheck used by testing/xvfb.py.

libjingle_media_unittest was missing a resource video in the .isolate
file.

BUG=chromium:497757
R=stip@chromium.org

Review URL: https://codereview.webrtc.org/1415603005 .

Cr-Commit-Position: refs/heads/master@{#10365}
---
 .gitignore                               |  1 +
 setup_links.py                           |  1 +
 talk/libjingle_media_unittest.isolate    |  1 +
 webrtc/modules/modules_unittests.isolate | 19 ++++++++++++++++++-
 webrtc/test/test.gyp                     |  5 +++++
 5 files changed, 26 insertions(+), 1 deletion(-)

diff --git a/.gitignore b/.gitignore
index 5b932c17b8..b29171c7d8 100644
--- a/.gitignore
+++ b/.gitignore
@@ -155,6 +155,7 @@
 /tools/valgrind
 /tools/vim
 /tools/win
+/tools/xdisplaycheck
 /tools/whitespace.txt
 /webrtc/examples/android/media_demo/bin
 /webrtc/examples/android/media_demo/gen
diff --git a/setup_links.py b/setup_links.py
index 99b526076c..9aeb1e8764 100755
--- a/setup_links.py
+++ b/setup_links.py
@@ -75,6 +75,7 @@ DIRECTORIES = [
   'tools/valgrind',
   'tools/vim',
   'tools/win',
+  'tools/xdisplaycheck',
 ]
 
 from sync_chromium import get_target_os_list
diff --git a/talk/libjingle_media_unittest.isolate b/talk/libjingle_media_unittest.isolate
index c250042dd6..2d67d0f969 100644
--- a/talk/libjingle_media_unittest.isolate
+++ b/talk/libjingle_media_unittest.isolate
@@ -34,6 +34,7 @@
         'files': [
           '<(PRODUCT_DIR)/libjingle_media_unittest<(EXECUTABLE_SUFFIX)',
           'media/testdata/captured-320x240-2s-48.frames',
+          'media/testdata/faces.1280x720_P420.yuv',
         ],
       },
     }],
diff --git a/webrtc/modules/modules_unittests.isolate b/webrtc/modules/modules_unittests.isolate
index bc6437f37f..5035f6ed0e 100644
--- a/webrtc/modules/modules_unittests.isolate
+++ b/webrtc/modules/modules_unittests.isolate
@@ -129,7 +129,7 @@
         ],
       },
     }],
-    ['OS=="linux" or OS=="mac" or OS=="win"', {
+    ['(OS=="linux" or OS=="mac" or OS=="win") and use_x11==0', {
       'variables': {
         'command': [
           '<(DEPTH)/testing/test_env.py',
@@ -142,5 +142,22 @@
         ],
       },
     }],
+    ['(OS=="linux" or OS=="mac" or OS=="win") and use_x11==1', {
+      'variables': {
+        'command': [
+          '<(DEPTH)/testing/xvfb.py',
+          '<(PRODUCT_DIR)',
+          '<(DEPTH)/testing/test_env.py',
+          '<(PRODUCT_DIR)/modules_unittests<(EXECUTABLE_SUFFIX)',
+        ],
+        'files': [
+          '<(DEPTH)/DEPS',
+          '<(DEPTH)/testing/test_env.py',
+          '<(DEPTH)/testing/xvfb.py',
+          '<(PRODUCT_DIR)/modules_unittests<(EXECUTABLE_SUFFIX)',
+          '<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
+        ],
+      },
+    }],
   ],
 }
diff --git a/webrtc/test/test.gyp b/webrtc/test/test.gyp
index a19288eaed..442b08723a 100644
--- a/webrtc/test/test.gyp
+++ b/webrtc/test/test.gyp
@@ -140,6 +140,11 @@
             'CLANG_ENABLE_OBJC_ARC': 'YES',
           },
         }],
+        ['use_x11==1', {
+          'dependencies': [
+            '<(DEPTH)/tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
+          ],
+        }],
       ],
     },
     {