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

Side by Side Diff: BUILD.gn

Issue 1379693002: GN: Remove irrelevant targets for Cast Android build (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 | « no previous file | 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 (c) 2013 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2013 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 is the root build file for GN. GN will start processing by loading this 5 # This is the root build file for GN. GN will start processing by loading this
6 # file, and recursively load all dependencies until all dependencies are either 6 # file, and recursively load all dependencies until all dependencies are either
7 # resolved or known not to exist (which will cause the build to fail). So if 7 # resolved or known not to exist (which will cause the build to fail). So if
8 # you add a new build file, there must be some path of dependencies from this 8 # you add a new build file, there must be some path of dependencies from this
9 # file to your new one or GN won't know about it. 9 # file to your new one or GN won't know about it.
10 10
(...skipping 196 matching lines...) Expand 10 before | Expand all | Expand 10 after
207 207
208 if (v8_use_external_startup_data) { 208 if (v8_use_external_startup_data) {
209 deps += [ "//gin:gin_v8_snapshot_fingerprint" ] 209 deps += [ "//gin:gin_v8_snapshot_fingerprint" ]
210 } 210 }
211 211
212 if (is_win) { 212 if (is_win) {
213 deps += [ "//chrome/tools/build/win/syzygy:chrome_dll_syzygy" ] 213 deps += [ "//chrome/tools/build/win/syzygy:chrome_dll_syzygy" ]
214 } 214 }
215 215
216 if (is_android) { 216 if (is_android) {
217 if (!is_chromecast) {
218 deps += [
219 "//chrome/android:chrome_public_apk",
220 "//chrome/android:chrome_public_test_apk",
221 "//chrome/test/chromedriver/test/webview_shell:chromedriver_webview_shel l_apk",
222 ]
223 }
Dirk Pranke 2015/09/29 22:35:59 it's a little weird to put the if block before the
224
217 deps += [ 225 deps += [
218 "//base/android/linker:chromium_android_linker", 226 "//base/android/linker:chromium_android_linker",
219 "//build/android/gyp/test:hello_world", 227 "//build/android/gyp/test:hello_world",
220 "//build/android/rezip", 228 "//build/android/rezip",
221 "//chrome/android:chrome_public_apk",
222 "//chrome/android:chrome_public_test_apk",
223 "//chrome/test/chromedriver/test/webview_shell:chromedriver_webview_shell_ apk",
224 "//third_party/errorprone:chromium_errorprone", 229 "//third_party/errorprone:chromium_errorprone",
225 "//tools/android:android_tools", 230 "//tools/android:android_tools",
226 "//tools/imagediff($host_toolchain)", 231 "//tools/imagediff($host_toolchain)",
227 232
228 # TODO(GYP): Remove these when the components_unittests work. 233 # TODO(GYP): Remove these when the components_unittests work.
229 "//components/history/core/test:test", 234 "//components/history/core/test:test",
230 "//components/policy:policy_component_test_support", 235 "//components/policy:policy_component_test_support",
231 "//components/policy:test_support", 236 "//components/policy:test_support",
232 "//components/rappor:test_support", 237 "//components/rappor:test_support",
233 "//components/signin/core/browser:test_support", 238 "//components/signin/core/browser:test_support",
(...skipping 744 matching lines...) Expand 10 before | Expand all | Expand 10 after
978 if (target_cpu == "x86") { 983 if (target_cpu == "x86") {
979 deps += [ 984 deps += [
980 # "//content/shell:crash_service_win64", TODO(GYP): crbug.com/537009 985 # "//content/shell:crash_service_win64", TODO(GYP): crbug.com/537009
981 ] 986 ]
982 } 987 }
983 } else { 988 } else {
984 deps += [ "//breakpad:minidump_stackwalk" ] 989 deps += [ "//breakpad:minidump_stackwalk" ]
985 } 990 }
986 } 991 }
987 } 992 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698