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

Side by Side Diff: BUILD.gn

Issue 1483843002: Port cronet build rules to GN part 1 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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 | build/gn_migration.gypi » ('j') | components/cronet/android/BUILD.gn » ('J')
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 294 matching lines...) Expand 10 before | Expand all | Expand 10 after
305 # TODO(GYP): Are these needed? 305 # TODO(GYP): Are these needed?
306 "//chrome/test:test_support_unit", 306 "//chrome/test:test_support_unit",
307 "//third_party/smhasher:murmurhash3", 307 "//third_party/smhasher:murmurhash3",
308 "//ui/message_center:test_support", 308 "//ui/message_center:test_support",
309 ] 309 ]
310 deps -= [ 310 deps -= [
311 "//net:net_perftests", 311 "//net:net_perftests",
312 "//url:url_unittests", 312 "//url:url_unittests",
313 ] 313 ]
314 314
315 if (!is_component_build) {
316 deps += [
317 "//components/cronet/android:cronet_sample_apk",
318 "//components/cronet/android:cronet_sample_test_apk",
319 "//components/cronet/android:cronet_test_apk",
320 "//components/cronet/android:cronet_test_instrumentation_apk",
321 "//components/cronet/android:cronet_unittests",
322 ]
323 }
324
315 if (!is_chromecast) { 325 if (!is_chromecast) {
316 deps += [ 326 deps += [
317 "//android_webview", 327 "//android_webview",
318 "//chrome/android:chrome_junit_tests", 328 "//chrome/android:chrome_junit_tests",
319 "//chrome/android:chrome_public_apk", 329 "//chrome/android:chrome_public_apk",
320 "//chrome/android:chrome_public_test_apk", 330 "//chrome/android:chrome_public_test_apk",
321 "//chrome/test/chromedriver/test/webview_shell:chromedriver_webview_shel l_apk", 331 "//chrome/test/chromedriver/test/webview_shell:chromedriver_webview_shel l_apk",
322 "//third_party/custom_tabs_client:custom_tabs_client_example_apk", 332 "//third_party/custom_tabs_client:custom_tabs_client_example_apk",
323 ] 333 ]
324 } 334 }
(...skipping 526 matching lines...) Expand 10 before | Expand all | Expand 10 after
851 if (target_cpu == "x86") { 861 if (target_cpu == "x86") {
852 deps += [ 862 deps += [
853 # "//content/shell:crash_service_win64", TODO(GYP): crbug.com/537009 863 # "//content/shell:crash_service_win64", TODO(GYP): crbug.com/537009
854 ] 864 ]
855 } 865 }
856 } else { 866 } else {
857 deps += [ "//breakpad:minidump_stackwalk($host_toolchain)" ] 867 deps += [ "//breakpad:minidump_stackwalk($host_toolchain)" ]
858 } 868 }
859 } 869 }
860 } 870 }
OLDNEW
« no previous file with comments | « no previous file | build/gn_migration.gypi » ('j') | components/cronet/android/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698