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

Unified Diff: chrome/test/data/nacl/nacl_test_data.gyp

Issue 14154005: Add dyncode syscall disabled test (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 8 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/test/data/nacl/nacl_test_data.gyp
diff --git a/chrome/test/data/nacl/nacl_test_data.gyp b/chrome/test/data/nacl/nacl_test_data.gyp
index fc43fd157dc61184174d217ec8a9f40c4738f379..8d70c6ef4a7bfcb50968bd077460347ef533f2de 100644
--- a/chrome/test/data/nacl/nacl_test_data.gyp
+++ b/chrome/test/data/nacl/nacl_test_data.gyp
@@ -127,6 +127,39 @@
'<(DEPTH)/native_client/tools.gyp:prep_toolchain',
]
},
+ {
+ 'target_name': 'pnacl_dyncode_syscall_disabled_test',
+ 'type': 'none',
+ 'variables': {
+ 'enable_x86_32': 0,
jvoung (off chromium) 2013/04/23 22:43:27 Maybe add a comment that these "enable_*: 0" are t
sehr 2013/04/24 15:34:25 Done.
+ 'enable_x86_64': 0,
+ 'enable_arm': 0,
bradn 2013/04/23 22:47:52 This just landed which will require you to change
sehr 2013/04/24 15:34:25 That change was reverted. So we're racing :-)
+ 'nexe_target': 'pnacl_dyncode_syscall_disabled',
+ 'build_pnacl_newlib': 1,
+ 'link_flags': [
+ '-lppapi',
bradn 2013/04/23 22:47:52 You probably don't need -lppapi after the rebase.
sehr 2013/04/24 15:34:25 Ditto.
+ '-lppapi_test_lib',
+ '-lplatform',
+ '-lgio',
+ '-lnacl_dyncode',
+ ],
+ 'sources': [
+ 'pnacl_dyncode_syscall_disabled/pnacl_dyncode_syscall_disabled.cc',
+ ],
+ 'test_files': [
+ 'pnacl_dyncode_syscall_disabled/pnacl_dyncode_syscall_disabled.html',
+ ],
+ },
+ 'dependencies': [
+ '<(DEPTH)/native_client/tools.gyp:prep_toolchain',
+ '<(DEPTH)/native_client/src/shared/platform/platform.gyp:platform_lib',
+ '<(DEPTH)/native_client/src/shared/gio/gio.gyp:gio_lib',
+ '<(DEPTH)/ppapi/native_client/native_client.gyp:ppapi_lib',
jvoung (off chromium) 2013/04/23 22:43:27 nit: sort this list?
sehr 2013/04/24 15:34:25 Done.
+ '<(DEPTH)/native_client/src/untrusted/nacl/nacl.gyp:nacl_dynacode_lib',
+ '<(DEPTH)/ppapi/ppapi_untrusted.gyp:ppapi_cpp_lib',
+ 'ppapi_test_lib',
+ ],
+ },
# Legacy NaCl PPAPI interface tests being here.
{
'target_name': 'ppapi_ppb_core',

Powered by Google App Engine
This is Rietveld 408576698