Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(359)

Unified Diff: chrome/chrome_tests.gypi

Issue 7011032: Make the gyp files more cross-platform across the Unices (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: updated to trunk and fixed some nits Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/chrome_tests.gypi
diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi
index 6bfe432d893ac768380ab07de50f75af60012e1c..c0b646905bef832e7874957af866ce5dc3ad949e 100644
--- a/chrome/chrome_tests.gypi
+++ b/chrome/chrome_tests.gypi
@@ -208,7 +208,7 @@
['exclude', '^browser/chromeos'],
],
}],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../build/linux/system.gyp:nss',
@@ -263,7 +263,7 @@
'chrome.gyp:crash_service', # run time dependency
],
}],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
],
@@ -348,7 +348,7 @@
'test/unit/run_all_unittests.cc',
],
'conditions': [
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
# Needed for the following #include chain:
# test/unit/run_all_unittests.cc
@@ -395,7 +395,7 @@
},
},
},],
- ['OS=="linux"', {
+ ['use_x11==1', {
'dependencies': [
'../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
],
@@ -469,14 +469,14 @@
'test/unit/chrome_test_suite.h',
],
'conditions': [
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../build/linux/system.gyp:nss',
'../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
],
}],
- ['OS=="linux" and toolkit_views==0', {
+ ['toolkit_uses_gtk==1 and toolkit_views==0', {
'sources!': [
# TODO(port)
'browser/ui/views/bookmarks/bookmark_bar_view_test.cc',
@@ -680,12 +680,12 @@
'../webkit/webkit.gyp:copy_npapi_test_plugin',
],
}],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
],
- }, { # else: OS != "linux"
+ }, { # else: toolkit_uses_gtk != 1
'sources!': [
'browser/process_singleton_linux_uitest.cc',
],
@@ -746,7 +746,7 @@
'browser/printing/printing_layout_uitest.cc',
],
}],
- ['OS=="linux" or OS=="freebsd"', {
+ ['os_posix==1 and OS!="mac"', {
'conditions': [
['linux_use_tcmalloc==1', {
'dependencies': [
@@ -864,7 +864,7 @@
'test/webdriver/commands/webelement_commands.cc',
],
'conditions': [
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
@@ -875,7 +875,7 @@
'../views/views.gyp:views',
],
}],
- ['OS=="linux" or OS=="freebsd"', {
+ ['os_posix==1 and OS!="mac"', {
'conditions': [
['linux_use_tcmalloc==1', {
'dependencies': [
@@ -1024,7 +1024,7 @@
],
},],
# Set fPIC in case it isn't set.
- ['(OS=="linux" or OS=="openbsd" or OS=="freebsd" or OS=="solaris")'
+ ['os_posix==1 and OS!="mac"'
'and (target_arch=="x64" or target_arch=="arm") and linux_fpic!=1', {
'cflags': ['-fPIC'],
},],
@@ -1990,7 +1990,7 @@
['exclude', '^browser/ui/webui/chromeos/login'],
],
}],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'conditions': [
['selinux==0', {
'dependencies': [
@@ -2020,14 +2020,14 @@
'sources!': [
'browser/printing/print_job_unittest.cc',
],
- }, { # else: OS != "linux"
+ }, { # else: toolkit_uses_gtk != 1
'sources!': [
'browser/ui/gtk/tabs/tab_renderer_gtk_unittest.cc',
'browser/renderer_host/gtk_key_bindings_handler_unittest.cc',
'../views/focus/accelerator_handler_gtk_unittest.cc',
],
}],
- ['OS=="linux" or OS=="freebsd"', {
+ ['os_posix==1 and OS!="mac"', {
'conditions': [
['linux_use_tcmalloc==1', {
'dependencies': [
@@ -2543,7 +2543,7 @@
'../content/browser/child_process_security_policy_browsertest.cc',
],
}],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../build/linux/system.gyp:nss',
@@ -2581,7 +2581,7 @@
'browser/extensions/browser_action_test_util_mac.mm',
],
}],
- ['OS=="linux" or OS=="freebsd"', {
+ ['os_posix==1 and OS!="mac"', {
'conditions': [
['linux_use_tcmalloc==1', {
'dependencies': [
@@ -2714,7 +2714,7 @@
},
},
},],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
@@ -2745,7 +2745,7 @@
},
},
},],
- ['OS=="linux" or OS=="freebsd"', {
+ ['os_posix==1 and OS!="mac"', {
'conditions': [
['linux_use_tcmalloc==1', {
'dependencies': [
@@ -2816,7 +2816,7 @@
},
},
},],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
],
@@ -2853,7 +2853,7 @@
},
},
},],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
@@ -2891,7 +2891,7 @@
'test/tab_switching/tab_switching_test.cc',
],
'conditions': [
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
@@ -2926,7 +2926,7 @@
'test/memory_test/memory_test.cc',
],
'conditions': [
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
@@ -3092,7 +3092,7 @@
'browser/sync/util/data_encryption_unittest.cc',
],
}],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../build/linux/system.gyp:nss',
@@ -3216,7 +3216,7 @@
'test/data/resource.rc',
],
'conditions': [
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../build/linux/system.gyp:nss',
@@ -3364,7 +3364,7 @@
# See comments about "xcode_settings" elsewhere in this file.
'xcode_settings': {'OTHER_LDFLAGS': ['-Wl,-ObjC']},
}],
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../build/linux/system.gyp:nss',
@@ -3499,7 +3499,7 @@
'test/perf/url_parse_perftest.cc',
],
'conditions': [
- ['OS=="linux"', {
+ ['toolkit_uses_gtk==1', {
'dependencies': [
'../build/linux/system.gyp:gtk',
'../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
@@ -3655,7 +3655,7 @@
], # OS=="win"
# If you change this condition, make sure you also change it in all.gyp
# for the chromium_builder_qa target.
- ['OS=="mac" or OS=="win" or (OS=="linux" and target_arch==python_arch)', {
+ ['OS=="mac" or OS=="win" or (os_posix==1 and target_arch==python_arch)', {
'targets': [
{
# Documentation: http://dev.chromium.org/developers/testing/pyauto
@@ -3707,7 +3707,7 @@
'GCC_TREAT_WARNINGS_AS_ERRORS': 'NO', # -Wno-error
},
'conditions': [
- ['OS=="linux"', {
+ ['os_posix==1 and OS!=="mac"', {
'include_dirs': [
'..',
'<(sysroot)/usr/include/python<(python_ver)',

Powered by Google App Engine
This is Rietveld 408576698