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

Side by Side Diff: chrome/chrome.gyp

Issue 159791: Move test_file_util out of libbase. Test code should not be mixed with production code. (Closed)
Patch Set: Created 11 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 unified diff | Download patch
« no previous file with comments | « base/base.gyp ('k') | net/net.gyp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'variables': { 6 'variables': {
7 'chromium_code': 1, 7 'chromium_code': 1,
8 # Define the common dependencies that contain all the actual 8 # Define the common dependencies that contain all the actual
9 # Chromium functionality. This list gets pulled in below by 9 # Chromium functionality. This list gets pulled in below by
10 # the link of the actual chrome (or chromium) executable on 10 # the link of the actual chrome (or chromium) executable on
(...skipping 3369 matching lines...) Expand 10 before | Expand all | Expand 10 after
3380 # production libraries but are used by more than one test executable. 3380 # production libraries but are used by more than one test executable.
3381 'target_name': 'test_support_common', 3381 'target_name': 'test_support_common',
3382 'type': '<(library)', 3382 'type': '<(library)',
3383 'dependencies': [ 3383 'dependencies': [
3384 'browser', 3384 'browser',
3385 'common', 3385 'common',
3386 'renderer', 3386 'renderer',
3387 'chrome_resources', 3387 'chrome_resources',
3388 'chrome_strings', 3388 'chrome_strings',
3389 'theme_resources', 3389 'theme_resources',
3390 '../base/base.gyp:test_support_base',
3390 '../skia/skia.gyp:skia', 3391 '../skia/skia.gyp:skia',
3391 '../testing/gmock.gyp:gmock', 3392 '../testing/gmock.gyp:gmock',
3392 '../testing/gtest.gyp:gtest', 3393 '../testing/gtest.gyp:gtest',
3393 ], 3394 ],
3394 'include_dirs': [ 3395 'include_dirs': [
3395 '..', 3396 '..',
3396 ], 3397 ],
3397 'sources': [ 3398 'sources': [
3398 # TODO: these should live here but are currently used by 3399 # TODO: these should live here but are currently used by
3399 # production code code in libbrowser (above). 3400 # production code code in libbrowser (above).
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
3468 ], 3469 ],
3469 }, 3470 },
3470 { 3471 {
3471 'target_name': 'test_support_ui', 3472 'target_name': 'test_support_ui',
3472 'type': '<(library)', 3473 'type': '<(library)',
3473 'dependencies': [ 3474 'dependencies': [
3474 'test_support_common', 3475 'test_support_common',
3475 'chrome_resources', 3476 'chrome_resources',
3476 'chrome_strings', 3477 'chrome_strings',
3477 'theme_resources', 3478 'theme_resources',
3479 '../base/base.gyp:test_support_base',
Mark Mentovai 2009/08/03 16:36:48 Is this really needed here? Isn't this already co
3478 '../skia/skia.gyp:skia', 3480 '../skia/skia.gyp:skia',
3479 '../testing/gtest.gyp:gtest', 3481 '../testing/gtest.gyp:gtest',
3480 ], 3482 ],
3481 'include_dirs': [ 3483 'include_dirs': [
3482 '..', 3484 '..',
3483 ], 3485 ],
3484 'sources': [ 3486 'sources': [
3485 'test/automated_ui_tests/automated_ui_test_base.cc', 3487 'test/automated_ui_tests/automated_ui_test_base.cc',
3486 'test/automated_ui_tests/automated_ui_test_base.h', 3488 'test/automated_ui_tests/automated_ui_test_base.h',
3487 'test/testing_browser_process.h', 3489 'test/testing_browser_process.h',
(...skipping 892 matching lines...) Expand 10 before | Expand all | Expand 10 after
4380 'tools/convert_dict/hunspell_reader.cc', 4382 'tools/convert_dict/hunspell_reader.cc',
4381 'tools/convert_dict/hunspell_reader.h', 4383 'tools/convert_dict/hunspell_reader.h',
4382 ], 4384 ],
4383 }, 4385 },
4384 { 4386 {
4385 'target_name': 'flush_cache', 4387 'target_name': 'flush_cache',
4386 'type': 'executable', 4388 'type': 'executable',
4387 'msvs_guid': '4539AFB3-B8DC-47F3-A491-6DAC8FD26657', 4389 'msvs_guid': '4539AFB3-B8DC-47F3-A491-6DAC8FD26657',
4388 'dependencies': [ 4390 'dependencies': [
4389 '../base/base.gyp:base', 4391 '../base/base.gyp:base',
4392 '../base/base.gyp:test_support_base',
4390 ], 4393 ],
4391 'sources': [ 4394 'sources': [
4392 'tools/perf/flush_cache/flush_cache.cc', 4395 'tools/perf/flush_cache/flush_cache.cc',
4393 ], 4396 ],
4394 }, 4397 },
4395 { 4398 {
4396 'target_name': 'pbl_tool', 4399 'target_name': 'pbl_tool',
4397 'type': 'executable', 4400 'type': 'executable',
4398 'msvs_guid': '265C427B-D1FF-4922-A47C-0754DDFF52E1', 4401 'msvs_guid': '265C427B-D1FF-4922-A47C-0754DDFF52E1',
4399 'dependencies': [ 4402 'dependencies': [
(...skipping 13 matching lines...) Expand all
4413 'msvs_guid': '9055E088-25C6-47FD-87D5-D9DD9FD75C9F', 4416 'msvs_guid': '9055E088-25C6-47FD-87D5-D9DD9FD75C9F',
4414 'dependencies': [ 4417 'dependencies': [
4415 'browser', 4418 'browser',
4416 'common', 4419 'common',
4417 'debugger', 4420 'debugger',
4418 'renderer', 4421 'renderer',
4419 'chrome_resources', 4422 'chrome_resources',
4420 'chrome_strings', 4423 'chrome_strings',
4421 '../base/base.gyp:base', 4424 '../base/base.gyp:base',
4422 '../base/base.gyp:test_support_base', 4425 '../base/base.gyp:test_support_base',
4426 '../base/base.gyp:test_support_perf',
4423 '../skia/skia.gyp:skia', 4427 '../skia/skia.gyp:skia',
4424 '../testing/gtest.gyp:gtest', 4428 '../testing/gtest.gyp:gtest',
4425 '../webkit/webkit.gyp:glue', 4429 '../webkit/webkit.gyp:glue',
4426 ], 4430 ],
4427 'sources': [ 4431 'sources': [
4428 'browser/safe_browsing/database_perftest.cc', 4432 'browser/safe_browsing/database_perftest.cc',
4429 'browser/safe_browsing/filter_false_positive_perftest.cc', 4433 'browser/safe_browsing/filter_false_positive_perftest.cc',
4430 'browser/visitedlink_perftest.cc', 4434 'browser/visitedlink_perftest.cc',
4431 'common/json_value_serializer_perftest.cc', 4435 'common/json_value_serializer_perftest.cc',
4432 'test/perf/perftests.cc', 4436 'test/perf/perftests.cc',
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
4466 { 4470 {
4467 # Executable that runs each browser test in a new process. 4471 # Executable that runs each browser test in a new process.
4468 'target_name': 'browser_tests', 4472 'target_name': 'browser_tests',
4469 'type': 'executable', 4473 'type': 'executable',
4470 'dependencies': [ 4474 'dependencies': [
4471 'browser', 4475 'browser',
4472 'chrome', 4476 'chrome',
4473 'chrome_resources', 4477 'chrome_resources',
4474 'debugger', 4478 'debugger',
4475 'test_support_common', 4479 'test_support_common',
4480 '../base/base.gyp:test_support_base',
4476 '../skia/skia.gyp:skia', 4481 '../skia/skia.gyp:skia',
4477 '../testing/gtest.gyp:gtest', 4482 '../testing/gtest.gyp:gtest',
4478 '../third_party/icu38/icu38.gyp:icui18n', 4483 '../third_party/icu38/icu38.gyp:icui18n',
4479 '../third_party/icu38/icu38.gyp:icuuc', 4484 '../third_party/icu38/icu38.gyp:icuuc',
4480 ], 4485 ],
4481 'include_dirs': [ 4486 'include_dirs': [
4482 '..', 4487 '..',
4483 ], 4488 ],
4484 'sources': [ 4489 'sources': [
4485 'test/browser/run_all_unittests.cc', 4490 'test/browser/run_all_unittests.cc',
(...skipping 336 matching lines...) Expand 10 before | Expand all | Expand 10 after
4822 'product_name': 'browser_tests', 4827 'product_name': 'browser_tests',
4823 'msvs_guid': 'D7589D0D-304E-4589-85A4-153B7D84B07F', 4828 'msvs_guid': 'D7589D0D-304E-4589-85A4-153B7D84B07F',
4824 'dependencies': [ 4829 'dependencies': [
4825 'chrome', 4830 'chrome',
4826 'browser', 4831 'browser',
4827 'chrome_dll_version', 4832 'chrome_dll_version',
4828 'chrome_resources', 4833 'chrome_resources',
4829 'installer/installer.gyp:installer_util_strings', 4834 'installer/installer.gyp:installer_util_strings',
4830 'debugger', 4835 'debugger',
4831 'renderer', 4836 'renderer',
4837 '../base/base.gyp:test_support_base',
4832 '../skia/skia.gyp:skia', 4838 '../skia/skia.gyp:skia',
4833 '../testing/gtest.gyp:gtest', 4839 '../testing/gtest.gyp:gtest',
4834 '../third_party/icu38/icu38.gyp:icui18n', 4840 '../third_party/icu38/icu38.gyp:icui18n',
4835 '../third_party/icu38/icu38.gyp:icuuc', 4841 '../third_party/icu38/icu38.gyp:icuuc',
4836 ], 4842 ],
4837 'include_dirs': [ 4843 'include_dirs': [
4838 '..', 4844 '..',
4839 'third_party/wtl/include', 4845 'third_party/wtl/include',
4840 ], 4846 ],
4841 'configurations': { 4847 'configurations': {
(...skipping 368 matching lines...) Expand 10 before | Expand all | Expand 10 after
5210 # Use outputs of this action as inputs for the main target build. 5216 # Use outputs of this action as inputs for the main target build.
5211 # Seems as a misnomer but makes this happy on Linux (scons). 5217 # Seems as a misnomer but makes this happy on Linux (scons).
5212 'process_outputs_as_sources': 1, 5218 'process_outputs_as_sources': 1,
5213 }, 5219 },
5214 ], # 'actions' 5220 ], # 'actions'
5215 }, 5221 },
5216 ] 5222 ]
5217 }], 5223 }],
5218 ], # 'conditions' 5224 ], # 'conditions'
5219 } 5225 }
OLDNEW
« no previous file with comments | « base/base.gyp ('k') | net/net.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698