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

Side by Side Diff: chrome/chrome_shell.gypi

Issue 864563002: Separate JNI registration with initialization (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address comments and sync 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 | « chrome/chrome_android.gypi ('k') | content/app/BUILD.gn » ('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 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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 GYPI allows independent customization of the chrome shell in a manner 5 # This GYPI allows independent customization of the chrome shell in a manner
6 # similar to content shell (in content_shell.gypi). Notably, this file does 6 # similar to content shell (in content_shell.gypi). Notably, this file does
7 # NOT contain chrome_android_core, which is independent of the chrome shell 7 # NOT contain chrome_android_core, which is independent of the chrome shell
8 # and should be separately customized. 8 # and should be separately customized.
9 { 9 {
10 'variables': { 10 'variables': {
(...skipping 30 matching lines...) Expand all
41 # Chrome shell should always use the statically-linked CLD data. 41 # Chrome shell should always use the statically-linked CLD data.
42 '<(DEPTH)/third_party/cld_2/cld_2.gyp:cld2_static', ], 42 '<(DEPTH)/third_party/cld_2/cld_2.gyp:cld2_static', ],
43 }], 43 }],
44 ], 44 ],
45 }, 45 },
46 { 46 {
47 # GN: //chrome/android:chrome_shell 47 # GN: //chrome/android:chrome_shell
48 'target_name': 'libchromeshell', 48 'target_name': 'libchromeshell',
49 'type': 'shared_library', 49 'type': 'shared_library',
50 'sources': [ 50 'sources': [
51 # This file must always be included in the shared_library step to ensure
52 # JNI_OnLoad is exported.
53 'app/android/chrome_jni_onload.cc',
54 'android/shell/chrome_main_delegate_chrome_shell_android.cc', 51 'android/shell/chrome_main_delegate_chrome_shell_android.cc',
55 'android/shell/chrome_main_delegate_chrome_shell_android.h', 52 'android/shell/chrome_main_delegate_chrome_shell_android.h',
56 ], 53 ],
57 'dependencies': [ 54 'dependencies': [
58 'libchromeshell_base', 55 'libchromeshell_base',
59 ], 56 ],
60 'includes': [ 57 'includes': [
61 # File 'protection' is based on non-trivial linker magic. TODO(pasko): 58 # File 'protection' is based on non-trivial linker magic. TODO(pasko):
62 # remove it when crbug.com/424562 is fixed. 59 # remove it when crbug.com/424562 is fixed.
63 '../base/files/protect_file_posix.gypi', 60 '../base/files/protect_file_posix.gypi',
64 ], 61 ],
65 }, 62 },
66 { 63 {
67 # GN: //chrome/android:chrome_sync_shell 64 # GN: //chrome/android:chrome_sync_shell
68 'target_name': 'libchromesyncshell', 65 'target_name': 'libchromesyncshell',
69 'type': 'shared_library', 66 'type': 'shared_library',
70 'sources': [ 67 'sources': [
71 # This file must always be included in the shared_library step to ensure
72 # JNI_OnLoad is exported.
73 'app/android/chrome_jni_onload.cc',
74 'android/sync_shell/chrome_main_delegate_chrome_sync_shell_android.cc', 68 'android/sync_shell/chrome_main_delegate_chrome_sync_shell_android.cc',
75 'android/sync_shell/chrome_main_delegate_chrome_sync_shell_android.h', 69 'android/sync_shell/chrome_main_delegate_chrome_sync_shell_android.h',
76 ], 70 ],
77 'dependencies': [ 71 'dependencies': [
78 'libchromeshell_base', 72 'libchromeshell_base',
79 '../sync/sync.gyp:test_support_sync_fake_server_android', 73 '../sync/sync.gyp:test_support_sync_fake_server_android',
80 ], 74 ],
81 }, 75 },
82 { 76 {
83 # GN: //chrome/android:chrome_shell_manifest 77 # GN: //chrome/android:chrome_shell_manifest
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
174 'target_name': 'chrome_sync_shell_apk_java', 168 'target_name': 'chrome_sync_shell_apk_java',
175 'type': 'none', 169 'type': 'none',
176 'dependencies': [ 170 'dependencies': [
177 'chrome_sync_shell_apk', 171 'chrome_sync_shell_apk',
178 ], 172 ],
179 'includes': [ '../build/apk_fake_jar.gypi' ], 173 'includes': [ '../build/apk_fake_jar.gypi' ],
180 }, 174 },
181 ], 175 ],
182 176
183 } 177 }
OLDNEW
« no previous file with comments | « chrome/chrome_android.gypi ('k') | content/app/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698