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

Side by Side Diff: base/base.gyp

Issue 2131423002: Implement use_cfi_cast to optionally enable cast checks. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Localize CFI_CAST_CHECK define Created 4 years, 5 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/BUILD.gn ('k') | base/tools_sanity_unittest.cc » ('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) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 }, 8 },
9 'includes': [ 9 'includes': [
10 '../build/win_precompile.gypi', 10 '../build/win_precompile.gypi',
(...skipping 621 matching lines...) Expand 10 before | Expand all | Expand 10 after
632 '../testing/gtest.gyp:gtest', 632 '../testing/gtest.gyp:gtest',
633 '../third_party/icu/icu.gyp:icui18n', 633 '../third_party/icu/icu.gyp:icui18n',
634 '../third_party/icu/icu.gyp:icuuc', 634 '../third_party/icu/icu.gyp:icuuc',
635 ], 635 ],
636 'includes': ['../build/nocompile.gypi'], 636 'includes': ['../build/nocompile.gypi'],
637 'variables': { 637 'variables': {
638 # TODO(ajwong): Is there a way to autodetect this? 638 # TODO(ajwong): Is there a way to autodetect this?
639 'module_dir': 'base' 639 'module_dir': 'base'
640 }, 640 },
641 'conditions': [ 641 'conditions': [
642 ['cfi_vptr==1 and cfi_cast==1', {
643 'defines': [
644 # TODO(krasin): remove CFI_CAST_CHECK, see https://crbug.com/626794 .
645 'CFI_CAST_CHECK',
646 ],
647 }],
642 ['OS == "ios" or OS == "mac"', { 648 ['OS == "ios" or OS == "mac"', {
643 'dependencies': [ 649 'dependencies': [
644 'base_unittests_arc', 650 'base_unittests_arc',
645 ], 651 ],
646 }], 652 }],
647 ['OS == "android"', { 653 ['OS == "android"', {
648 'dependencies': [ 654 'dependencies': [
649 'android/jni_generator/jni_generator.gyp:jni_generator_tests', 655 'android/jni_generator/jni_generator.gyp:jni_generator_tests',
650 '../testing/android/native_test.gyp:native_test_native_code', 656 '../testing/android/native_test.gyp:native_test_native_code',
651 ], 657 ],
(...skipping 1134 matching lines...) Expand 10 before | Expand all | Expand 10 after
1786 ['include', 'mac/bind_objc_block_unittest_arc\\.mm$'], 1792 ['include', 'mac/bind_objc_block_unittest_arc\\.mm$'],
1787 ['include', 'mac/scoped_nsobject_unittest_arc\\.mm$'], 1793 ['include', 'mac/scoped_nsobject_unittest_arc\\.mm$'],
1788 ], 1794 ],
1789 }] 1795 }]
1790 ], 1796 ],
1791 }, 1797 },
1792 ], 1798 ],
1793 }], 1799 }],
1794 ], 1800 ],
1795 } 1801 }
OLDNEW
« no previous file with comments | « base/BUILD.gn ('k') | base/tools_sanity_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698