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

Side by Side Diff: chrome/chrome_shell.gypi

Issue 657443002: Turn Chrome Shell's AndroidManifest into a jinja2 template. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: ignore import failure in pylint Created 6 years, 2 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/android/shell/java/AndroidManifest.xml ('k') | no next file » | 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 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 'app/android/chrome_jni_onload.cc', 76 'app/android/chrome_jni_onload.cc',
77 'android/sync_shell/chrome_main_delegate_chrome_sync_shell_android.cc', 77 'android/sync_shell/chrome_main_delegate_chrome_sync_shell_android.cc',
78 'android/sync_shell/chrome_main_delegate_chrome_sync_shell_android.h', 78 'android/sync_shell/chrome_main_delegate_chrome_sync_shell_android.h',
79 ], 79 ],
80 'dependencies': [ 80 'dependencies': [
81 'libchromeshell_base', 81 'libchromeshell_base',
82 '../sync/sync.gyp:test_support_sync_fake_server_android', 82 '../sync/sync.gyp:test_support_sync_fake_server_android',
83 ], 83 ],
84 }, 84 },
85 { 85 {
86 # GN: //chrome/android:chrome_shell_manifest
87 'target_name': 'chrome_shell_manifest',
88 'type': 'none',
89 'variables': {
90 'jinja_inputs': ['android/shell/java/AndroidManifest.xml'],
91 'jinja_output': '<(SHARED_INTERMEDIATE_DIR)/chrome_shell_manifest/Androi dManifest.xml',
92 },
93 'includes': [ '../build/android/jinja_template.gypi' ],
94 },
95 {
86 # GN: //chrome/android:chrome_shell_apk 96 # GN: //chrome/android:chrome_shell_apk
87 'target_name': 'chrome_shell_apk', 97 'target_name': 'chrome_shell_apk',
88 'type': 'none', 98 'type': 'none',
89 'dependencies': [ 99 'dependencies': [
90 'chrome_java', 100 'chrome_java',
91 'chrome_shell_paks', 101 'chrome_shell_paks',
92 'libchromeshell', 102 'libchromeshell',
93 '../media/media.gyp:media_java', 103 '../media/media.gyp:media_java',
94 ], 104 ],
95 'variables': { 105 'variables': {
96 'apk_name': 'ChromeShell', 106 'apk_name': 'ChromeShell',
107 'android_manifest_path': '<(SHARED_INTERMEDIATE_DIR)/chrome_shell_manife st/AndroidManifest.xml',
97 'native_lib_version_name': '<(version_full)', 108 'native_lib_version_name': '<(version_full)',
98 'java_in_dir': 'android/shell/java', 109 'java_in_dir': 'android/shell/java',
99 'resource_dir': 'android/shell/res', 110 'resource_dir': 'android/shell/res',
100 'asset_location': '<(PRODUCT_DIR)/../assets/<(package_name)', 111 'asset_location': '<(PRODUCT_DIR)/../assets/<(package_name)',
101 'native_lib_target': 'libchromeshell', 112 'native_lib_target': 'libchromeshell',
102 'additional_input_paths': [ 113 'additional_input_paths': [
103 '<@(chrome_android_pak_output_resources)', 114 '<@(chrome_android_pak_output_resources)',
104 ], 115 ],
105 }, 116 },
106 'includes': [ '../build/java_apk.gypi', ], 117 'includes': [ '../build/java_apk.gypi', ],
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 'target_name': 'chrome_sync_shell_apk_java', 183 'target_name': 'chrome_sync_shell_apk_java',
173 'type': 'none', 184 'type': 'none',
174 'dependencies': [ 185 'dependencies': [
175 'chrome_sync_shell_apk', 186 'chrome_sync_shell_apk',
176 ], 187 ],
177 'includes': [ '../build/apk_fake_jar.gypi' ], 188 'includes': [ '../build/apk_fake_jar.gypi' ],
178 }, 189 },
179 ], 190 ],
180 191
181 } 192 }
OLDNEW
« no previous file with comments | « chrome/android/shell/java/AndroidManifest.xml ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698