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

Side by Side Diff: base/base.gyp

Issue 1224963002: Refactor chromium's Linker class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update for review feedback. Created 5 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
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 1517 matching lines...) Expand 10 before | Expand all | Expand 10 after
1528 'variables': { 1528 'variables': {
1529 'java_in_dir': '../base/android/javatests', 1529 'java_in_dir': '../base/android/javatests',
1530 }, 1530 },
1531 'includes': [ '../build/java.gypi' ], 1531 'includes': [ '../build/java.gypi' ],
1532 }, 1532 },
1533 { 1533 {
1534 # GN: //base/android/linker:chromium_android_linker 1534 # GN: //base/android/linker:chromium_android_linker
1535 'target_name': 'chromium_android_linker', 1535 'target_name': 'chromium_android_linker',
1536 'type': 'shared_library', 1536 'type': 'shared_library',
1537 'sources': [ 1537 'sources': [
1538 'android/linker/linker_jni.cc', 1538 'android/linker/legacy_linker_jni.cc',
1539 ], 1539 ],
1540 # The crazy linker is never instrumented. 1540 # The crazy linker is never instrumented.
1541 'cflags!': [ 1541 'cflags!': [
1542 '-finstrument-functions', 1542 '-finstrument-functions',
1543 ], 1543 ],
1544 'dependencies': [ 1544 'dependencies': [
1545 # The NDK contains the crazy_linker here: 1545 # The NDK contains the crazy_linker here:
1546 # '<(android_ndk_root)/crazy_linker.gyp:crazy_linker' 1546 # '<(android_ndk_root)/crazy_linker.gyp:crazy_linker'
1547 # However, we use our own fork. See bug 384700. 1547 # However, we use our own fork. See bug 384700.
1548 '../third_party/android_crazy_linker/crazy_linker.gyp:crazy_linker', 1548 '../third_party/android_crazy_linker/crazy_linker.gyp:crazy_linker',
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
1626 '../build/isolate.gypi', 1626 '../build/isolate.gypi',
1627 ], 1627 ],
1628 'sources': [ 1628 'sources': [
1629 'base_unittests.isolate', 1629 'base_unittests.isolate',
1630 ], 1630 ],
1631 }, 1631 },
1632 ], 1632 ],
1633 }], 1633 }],
1634 ], 1634 ],
1635 } 1635 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698