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

Unified Diff: chrome/chrome_tests.gypi

Issue 285283003: Remove flag --test-sandbox (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: change requested by resek, thanks Created 6 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
« no previous file with comments | « chrome/browser_tests.isolate ('k') | chrome/test/security_tests/ipc_security_tests.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_tests.gypi
diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi
index 16e8531d41a16ce4981860609a8e960e27b088a0..4701982307c61a0127797d22f32f1eb4561750fb 100644
--- a/chrome/chrome_tests.gypi
+++ b/chrome/chrome_tests.gypi
@@ -1595,8 +1595,6 @@
'test/remoting/remote_desktop_browsertest.h',
'test/remoting/waiter.cc',
'test/remoting/waiter.h',
- 'test/security_tests/sandbox_browsertest_linux.cc',
- 'test/security_tests/sandbox_browsertest_win.cc',
# TODO(craig): Rename this and run from base_unittests when the test
# is safe to run there. See http://crbug.com/78722 for details.
'../base/files/file_path_watcher_browsertest.cc',
@@ -1825,7 +1823,6 @@
],
'dependencies': [
'chrome_version_resources',
- 'security_tests', # run time dependency
],
'conditions': [
['win_use_allocator_shim==1', {
@@ -2604,31 +2601,6 @@
['OS=="mac"', {
'targets': [
{
- # This is the mac equivalent of the security_tests target below. It
- # generates a framework bundle which bundles tests to be run in a
- # renderer process. The test code is built as a framework so it can be
- # run in the context of a renderer without shipping the code to end
- # users.
- 'target_name': 'renderer_sandbox_tests',
- 'type': 'shared_library',
- 'product_name': 'Renderer Sandbox Tests',
- 'mac_bundle': 1,
- 'xcode_settings': {
- 'INFOPLIST_FILE': 'test/security_tests/sandbox_tests_mac-Info.plist',
- },
- 'sources': [
- 'test/security_tests/renderer_sandbox_tests_mac.mm',
- ],
- 'include_dirs': [
- '..',
- ],
- 'link_settings': {
- 'libraries': [
- '$(SDKROOT)/System/Library/Frameworks/Cocoa.framework',
- ],
- },
- }, # target renderer_sandbox_tests
- {
# Tests for Mac app launcher.
'target_name': 'app_mode_app_tests',
'type': 'executable',
@@ -2727,24 +2699,6 @@
},
],
},], # OS!="mac"
- ['OS=="win"', {
- 'targets': [
- {
- 'target_name': 'security_tests',
- 'type': 'shared_library',
- 'include_dirs': [
- '..',
- ],
- 'sources': [
- 'test/security_tests/ipc_security_tests.cc',
- 'test/security_tests/ipc_security_tests.h',
- 'test/security_tests/security_tests.cc',
- '../sandbox/win/tests/validation_tests/commands.cc',
- '../sandbox/win/tests/validation_tests/commands.h',
- ],
- },
- ]}, # 'targets'
- ], # OS=="win"
['OS == "android"', {
'targets': [
{
« no previous file with comments | « chrome/browser_tests.isolate ('k') | chrome/test/security_tests/ipc_security_tests.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698