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

Unified Diff: build/common.gypi

Issue 68193002: Remove linux coverage scripts. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 1 month 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 | « base/base.gyp ('k') | chrome/chrome_exe.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index d0f9f22fb820c780c0068e4394b1e22b0c719079..9a8cc7e5da4909b32a4293015e3f86f739d8e5d9 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -953,12 +953,6 @@
# set there, per the comment about variable% in a target_defaults.
'mac_strip_release%': 0,
- # Set to 1 to enable code coverage. In addition to build changes
- # (e.g. extra CFLAGS), also creates a new target in the src/chrome
- # project file called "coverage".
- # Currently ignored on Windows.
- 'coverage%': 0,
-
# Set to 1 to enable java code coverage. Instruments classes during build
# to produce .ec files during runtime.
'emma_coverage%': 0,
@@ -2226,55 +2220,6 @@
'MEMORY_TOOL_REPLACES_ALLOCATOR',
],
}], # asan==1 and OS=="win"
- ['coverage!=0', {
- 'conditions': [
- ['OS=="mac" or OS=="ios"', {
- 'xcode_settings': {
- 'GCC_INSTRUMENT_PROGRAM_FLOW_ARCS': 'YES', # -fprofile-arcs
- 'GCC_GENERATE_TEST_COVERAGE_FILES': 'YES', # -ftest-coverage
- },
- }],
- ['OS=="mac"', {
- # Add -lgcov for types executable, shared_library, and
- # loadable_module; not for static_library.
- # This is a delayed conditional.
- 'target_conditions': [
- ['_type!="static_library"', {
- 'xcode_settings': { 'OTHER_LDFLAGS': [ '-lgcov' ] },
- }],
- ],
- }],
- ['OS=="linux" or OS=="android"', {
- 'cflags': [ '-ftest-coverage',
- '-fprofile-arcs' ],
- 'link_settings': { 'libraries': [ '-lgcov' ] },
- }],
- ['OS=="win"', {
- 'variables': {
- # Disable incremental linking for all modules.
- # 0: inherit, 1: disabled, 2: enabled.
- 'msvs_debug_link_incremental': '1',
- 'msvs_large_module_debug_link_mode': '1',
- # Disable RTC. Syzygy explicitly doesn't support RTC instrumented
- # binaries for now.
- 'win_debug_RuntimeChecks': '0',
- },
- 'defines': [
- # Disable iterator debugging (huge speed boost without any
- # change in coverage results).
- '_HAS_ITERATOR_DEBUGGING=0',
- ],
- 'msvs_settings': {
- 'VCLinkerTool': {
- # Enable profile information (necessary for coverage
- # instrumentation). This is incompatible with incremental
- # linking.
- 'Profile': 'true',
- },
- }
- }], # OS==win
- ], # conditions for coverage
- }], # coverage!=0
['OS=="win"', {
'defines': [
'__STD_C',
« no previous file with comments | « base/base.gyp ('k') | chrome/chrome_exe.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698