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

Side by Side Diff: build/jar_file_jni_generator.gypi

Issue 920883002: Use combined native/manual JNI registration. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Factor out common code in test Created 5 years, 10 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 | « build/config/android/rules.gni ('k') | build/jni_generator.gypi » ('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 # This file is meant to be included into a target to provide a rule 5 # This file is meant to be included into a target to provide a rule
6 # to generate jni bindings for system Java-files in a consistent manner. 6 # to generate jni bindings for system Java-files in a consistent manner.
7 # 7 #
8 # To use this, create a gyp target with the following form: 8 # To use this, create a gyp target with the following form:
9 # { 9 # {
10 # 'target_name': 'android_jar_jni_headers', 10 # 'target_name': 'android_jar_jni_headers',
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 'hard_dependency': 1, 66 'hard_dependency': 1,
67 'conditions': [ 67 'conditions': [
68 ['android_webview_build==1', { 68 ['android_webview_build==1', {
69 'variables': { 69 'variables': {
70 'native_exports%': '--native_exports', 70 'native_exports%': '--native_exports',
71 }, 71 },
72 'dependencies': [ 72 'dependencies': [
73 '<(DEPTH)/build/android/android_exports.gyp:android_exports', 73 '<(DEPTH)/build/android/android_exports.gyp:android_exports',
74 ], 74 ],
75 }], 75 }],
76 ['clang==0', {
77 # Clang builds currently fail with --native_exports_optional due to
78 # http://llvm.org/bugs/show_bug.cgi?id=22602 - only enable for gcc.
79 # http://crbug.com/442327
80 'variables': {
81 'native_exports%': '--native_exports_optional',
82 },
83 }],
76 ], 84 ],
77 } 85 }
OLDNEW
« no previous file with comments | « build/config/android/rules.gni ('k') | build/jni_generator.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698