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

Unified Diff: ppapi/native_client/tests/ppapi_browser/crash/nacl.scons

Issue 7778012: Landing chrome side of nacl change pre-dating ppapi switch. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 4 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: ppapi/native_client/tests/ppapi_browser/crash/nacl.scons
===================================================================
--- ppapi/native_client/tests/ppapi_browser/crash/nacl.scons (revision 98496)
+++ ppapi/native_client/tests/ppapi_browser/crash/nacl.scons (working copy)
@@ -11,10 +11,13 @@
Import('env')
env.Prepend(CPPDEFINES=['XP_UNIX'])
+env.Replace(
+ TEST_DIR='${SOURCE_ROOT}/ppapi/native_client/tests/ppapi_browser/crash')
crash_types = ['via_check_failure', 'via_exit_call', 'in_callback']
published_files = []
+nmfs = []
for crash_type in crash_types:
nexe = 'ppapi_crash_%s_%s' % (crash_type, env.get('TARGET_FULLARCH'))
env.Alias('ppapi_crash_%s${PROGSUFFIX}' % crash_type,
@@ -32,17 +35,19 @@
# TODO(polina): follow ppapi_browser/bad nacl.scons to publish html only once
# and to associate all nexes and nmf files with one target that can be
# referenced in tests/nacl.scons.
+ nmfs.append('${TEST_DIR}/ppapi_crash_%s.nmf' % crash_type)
env.Publish(nexe, 'run',
- ['ppapi_crash.html',
- 'ppapi_crash_%s.nmf' % crash_type ])
+ ['ppapi_crash.html'])
published_files.extend(env.ExtractPublishedFiles(nexe))
node = env.PPAPIBrowserTester('ppapi_crash_browser_test.out',
url='ppapi_crash.html',
+ nmf=nmfs,
files=published_files)
env.AddNodeToTestSuite(node,
['chrome_browser_tests'],
'run_ppapi_crash_browser_test',
- is_broken=env.PPAPIBrowserTesterIsBroken())
+ is_broken=env.PPAPIBrowserTesterIsBroken() or
+ env.Bit('nacl_glibc'))

Powered by Google App Engine
This is Rietveld 408576698