0

Use Xvfb in isolated testing only if use_x11 is set

We don't need a virtual X server for ozone testing, and it is causing
problems running the tests with swarming. This changes all isolate files
to only run Xvfb if use_x11==1 is set in GYP_DEFINES, and merges ozone
with Windows & Mac (none of which need to run their own display server).

BUG=440882
TEST=isolate.py run -s out_ozone/Debug/<various>
TBR=maruel@chromium.org

Review URL: https://codereview.chromium.org/813363003

Cr-Commit-Position: refs/heads/master@{#310434}
This commit is contained in:
spang
2015-01-07 17:42:06 -08:00
committed by Commit bot
parent 3e81e58f70
commit 8a7be07148
15 changed files with 217 additions and 221 deletions

@ -15,7 +15,16 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/ash_unittests<(EXECUTABLE_SUFFIX)',
'--test-launcher-bot-mode',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../testing/xvfb.py',
@ -25,25 +34,20 @@
],
'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', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="win"', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/ash_unittests.exe',
'--test-launcher-bot-mode',
],
'files': [
'<(PRODUCT_DIR)/ffmpegsumo.dll',
'<(PRODUCT_DIR)/osmesa.dll',

@ -10,7 +10,19 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/base_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../testing/xvfb.py',
@ -23,12 +35,6 @@
],
'files': [
'../testing/xvfb.py',
],
},
}],
['OS=="linux" and use_ozone==0', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
@ -42,18 +48,6 @@
'read_only': 1,
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/base_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['OS=="win" and (fastbuild==0 or fastbuild==1)', {
'variables': {
'files': [

@ -37,6 +37,7 @@ def DefaultConfigVariables():
'target_arch': 'arm',
'use_openssl': '0',
'use_ozone': '0',
'use_x11': '0',
'v8_use_external_startup_data': '0',
}

@ -91,6 +91,7 @@
'--config-variable', 'use_openssl=<(use_openssl)',
'--config-variable', 'target_arch=<(target_arch)',
'--config-variable', 'use_ozone=<(use_ozone)',
'--config-variable', 'use_x11=<(use_x11)',
'--config-variable', 'disable_nacl=<(disable_nacl)',
],
'conditions': [

@ -3,7 +3,18 @@
# found in the LICENSE file.
{
'conditions': [
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/browser_tests<(EXECUTABLE_SUFFIX)',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../testing/xvfb.py',
@ -15,16 +26,16 @@
],
'files': [
'../testing/xvfb.py',
'<(PRODUCT_DIR)/libclearkeycdm.so',
'<(PRODUCT_DIR)/libclearkeycdmadapter.so',
'<(PRODUCT_DIR)/libppapi_tests.so',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" and use_ozone==0', {
['OS=="linux"', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/libclearkeycdm.so',
'<(PRODUCT_DIR)/libclearkeycdmadapter.so',
'<(PRODUCT_DIR)/libppapi_tests.so',
],
},
}],
@ -158,17 +169,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/browser_tests<(EXECUTABLE_SUFFIX)',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['OS=="win"', {
'variables': {
'files': [

@ -3,7 +3,18 @@
# found in the LICENSE file.
{
'conditions': [
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/interactive_ui_tests<(EXECUTABLE_SUFFIX)',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../testing/xvfb.py',
@ -15,6 +26,13 @@
],
'files': [
'../testing/xvfb.py',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux"', {
'variables': {
'files': [
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/libosmesa.so',
'<(PRODUCT_DIR)/libppapi_tests.so',
@ -22,13 +40,6 @@
],
},
}],
['OS=="linux" and use_ozone==0', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" or OS=="mac" or OS=="win"', {
'variables': {
'files': [
@ -68,17 +79,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/interactive_ui_tests<(EXECUTABLE_SUFFIX)',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['OS=="win"', {
'variables': {
'files': [

@ -3,7 +3,15 @@
# found in the LICENSE file.
{
'conditions': [
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'<(PRODUCT_DIR)/sync_integration_tests<(EXECUTABLE_SUFFIX)',
'--test-launcher-bot-mode',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../testing/xvfb.py',
@ -14,15 +22,15 @@
'files': [
'../testing/test_env.py',
'../testing/xvfb.py',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/libosmesa.so',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" and use_ozone==0', {
['OS=="linux"', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/libosmesa.so',
],
},
}],
@ -49,14 +57,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'<(PRODUCT_DIR)/sync_integration_tests<(EXECUTABLE_SUFFIX)',
'--test-launcher-bot-mode',
],
},
}],
['OS=="mac"', {
'variables': {
'files': [

@ -39,7 +39,19 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/unit_tests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../testing/xvfb.py',
@ -52,16 +64,16 @@
],
'files': [
'../testing/xvfb.py',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/libosmesa.so',
'<(PRODUCT_DIR)/locales/fr.pak',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" and use_ozone==0', {
['OS=="linux"', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/libosmesa.so',
'<(PRODUCT_DIR)/locales/fr.pak',
],
},
}],
@ -93,18 +105,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/unit_tests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['OS=="win"', {
'variables': {
'files': [

@ -33,7 +33,18 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/content_browsertests<(EXECUTABLE_SUFFIX)',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../testing/xvfb.py',
@ -45,6 +56,13 @@
],
'files': [
'../testing/xvfb.py',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux"', {
'variables': {
'files': [
'<(PRODUCT_DIR)/content_shell.pak',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/libosmesa.so',
@ -52,13 +70,6 @@
],
},
}],
['OS=="linux" and use_ozone==0', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" or OS=="mac" or OS=="win"', {
'variables': {
'files': [
@ -84,17 +95,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/content_browsertests<(EXECUTABLE_SUFFIX)',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['OS=="win"', {
'variables': {
'files': [

@ -33,7 +33,19 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/content_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../testing/xvfb.py',
@ -46,28 +58,20 @@
],
'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', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="mac"', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/content_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
'files': [
'<(PRODUCT_DIR)/ffmpegsumo.so',
'<(PRODUCT_DIR)/osmesa.so',
@ -76,14 +80,6 @@
}],
['OS=="win"', {
'variables': {
'command': [
'../testing/test_env.py',
'<(PRODUCT_DIR)/content_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
'files': [
'<(PRODUCT_DIR)/ffmpegsumo.dll',
'<(PRODUCT_DIR)/osmesa.dll',

@ -12,7 +12,19 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/app_list_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../../testing/xvfb.py',
@ -25,14 +37,14 @@
],
'files': [
'../../testing/xvfb.py',
'<(PRODUCT_DIR)/libosmesa.so',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" and use_ozone==0', {
['OS=="linux"', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/libosmesa.so',
],
},
}],
@ -50,18 +62,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/app_list_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
],
'includes': [
'../../base/base.isolate',

@ -21,7 +21,19 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/ui_base_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../../testing/xvfb.py',
@ -34,14 +46,14 @@
],
'files': [
'../../testing/xvfb.py',
'<(PRODUCT_DIR)/libosmesa.so',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" and use_ozone==0', {
['OS=="linux"', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/libosmesa.so',
],
},
}],
@ -60,18 +72,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/ui_base_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
],
'includes': [
'../../base/base.isolate',

@ -11,7 +11,19 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/events_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../../testing/xvfb.py',
@ -24,14 +36,14 @@
],
'files': [
'../../testing/xvfb.py',
'<(PRODUCT_DIR)/libosmesa.so',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" and use_ozone==0', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/libosmesa.so',
],
},
}],
@ -49,18 +61,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/events_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
],
'includes': [
'../../base/base.isolate',

@ -12,7 +12,19 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/message_center_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../../testing/xvfb.py',
@ -25,14 +37,14 @@
],
'files': [
'../../testing/xvfb.py',
'<(PRODUCT_DIR)/libosmesa.so',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" and use_ozone==0', {
['OS=="linux"', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/libosmesa.so',
],
},
}],
@ -50,18 +62,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/message_center_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
],
'includes': [
'../../base/base.isolate',

@ -11,7 +11,19 @@
],
},
}],
['OS=="linux"', {
['use_x11==0', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/ui_touch_selection_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
['use_x11==1', {
'variables': {
'command': [
'../../testing/xvfb.py',
@ -24,14 +36,14 @@
],
'files': [
'../../testing/xvfb.py',
'<(PRODUCT_DIR)/libosmesa.so',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
],
},
}],
['OS=="linux" and use_ozone==0', {
['OS=="linux"', {
'variables': {
'files': [
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/libosmesa.so',
],
},
}],
@ -49,18 +61,6 @@
],
},
}],
['OS=="mac" or OS=="win"', {
'variables': {
'command': [
'../../testing/test_env.py',
'<(PRODUCT_DIR)/ui_touch_selection_unittests<(EXECUTABLE_SUFFIX)',
'--brave-new-test-launcher',
'--test-launcher-bot-mode',
'--asan=<(asan)',
'--lsan=<(lsan)',
],
},
}],
],
'includes': [
'../../base/base.isolate',