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

Side by Side Diff: BUILD.gn

Issue 1446453004: Adding third_party/openh264 build files for encoding (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use_openh264 = conditional on chrome branding again Created 5 years, 1 month 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 | « .gn ('k') | build/all.gyp » ('j') | third_party/openh264/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
11 import("//build/config/features.gni") 11 import("//build/config/features.gni")
12 import("//build/config/sanitizers/sanitizers.gni") 12 import("//build/config/sanitizers/sanitizers.gni")
13 import("//build/config/ui.gni") 13 import("//build/config/ui.gni")
14 import("//build_overrides/v8.gni") 14 import("//build_overrides/v8.gni")
15 import("//media/media_options.gni") 15 import("//media/media_options.gni")
16 import("//third_party/openh264/openh264_args.gni")
16 17
17 if (is_android) { 18 if (is_android) {
18 import("//build/config/android/config.gni") 19 import("//build/config/android/config.gni")
19 } 20 }
20 21
21 declare_args() { 22 declare_args() {
22 # A list of extra dependencies to add to the root target. This allows a 23 # A list of extra dependencies to add to the root target. This allows a
23 # checkout to add additional targets without explicitly changing any checked- 24 # checkout to add additional targets without explicitly changing any checked-
24 # in files. 25 # in files.
25 root_extra_deps = [] 26 root_extra_deps = []
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 group("both_gn_and_gyp") { 86 group("both_gn_and_gyp") {
86 testonly = true 87 testonly = true
87 deps = [ 88 deps = [
88 "//base:base_unittests", 89 "//base:base_unittests",
89 "//chrome/installer", 90 "//chrome/installer",
90 "//components:components_unittests", 91 "//components:components_unittests",
91 "//net:net_unittests", 92 "//net:net_unittests",
92 "//skia:skia_unittests", 93 "//skia:skia_unittests",
93 "//sql:sql_unittests", 94 "//sql:sql_unittests",
94 "//sync:sync_unit_tests", 95 "//sync:sync_unit_tests",
96 "//third_party/openh264/testing:openh264_unittests",
Dirk Pranke 2015/11/21 00:48:51 Are you sure you want to add this unconditionally,
hbos_chromium 2015/11/23 09:06:13 If we can make trybots conditionally run the binar
95 "//ui/base:ui_base_unittests", 97 "//ui/base:ui_base_unittests",
96 "//ui/gfx:gfx_unittests", 98 "//ui/gfx:gfx_unittests",
97 "//url:url_unittests", 99 "//url:url_unittests",
98 ] 100 ]
99 101
100 if (!is_ios && !is_android && !is_chromecast) { 102 if (!is_ios && !is_android && !is_chromecast) {
101 deps += [ 103 deps += [
102 "//chrome", 104 "//chrome",
103 "//chrome/test:browser_tests", 105 "//chrome/test:browser_tests",
104 "//chrome/test:interactive_ui_tests", 106 "//chrome/test:interactive_ui_tests",
(...skipping 474 matching lines...) Expand 10 before | Expand all | Expand 10 after
579 ] 581 ]
580 } else if (!is_android && !is_ios) { 582 } else if (!is_android && !is_ios) {
581 deps += [ 583 deps += [
582 "//breakpad:symupload($host_toolchain)", 584 "//breakpad:symupload($host_toolchain)",
583 "//media/cast:cast_unittests", 585 "//media/cast:cast_unittests",
584 ] 586 ]
585 } 587 }
586 if (is_chromecast) { 588 if (is_chromecast) {
587 deps += [ "//chromecast:cast_shell" ] 589 deps += [ "//chromecast:cast_shell" ]
588 } 590 }
591
592 if (use_openh264) {
593 deps += [
594 "//third_party/openh264:openh264_common",
595 "//third_party/openh264:openh264_processing",
596 "//third_party/openh264:openh264_encoder",
597 "//third_party/openh264:openh264_decoder",
598 ]
599 }
589 } 600 }
590 601
591 group("gn_only") { 602 group("gn_only") {
592 testonly = true 603 testonly = true
593 604
594 deps = [] 605 deps = []
595 606
596 if (!is_ios && !is_chromecast) { 607 if (!is_ios && !is_chromecast) {
597 deps += [ "//mandoline:all" ] 608 deps += [ "//mandoline:all" ]
598 } 609 }
(...skipping 241 matching lines...) Expand 10 before | Expand all | Expand 10 after
840 if (target_cpu == "x86") { 851 if (target_cpu == "x86") {
841 deps += [ 852 deps += [
842 # "//content/shell:crash_service_win64", TODO(GYP): crbug.com/537009 853 # "//content/shell:crash_service_win64", TODO(GYP): crbug.com/537009
843 ] 854 ]
844 } 855 }
845 } else { 856 } else {
846 deps += [ "//breakpad:minidump_stackwalk($host_toolchain)" ] 857 deps += [ "//breakpad:minidump_stackwalk($host_toolchain)" ]
847 } 858 }
848 } 859 }
849 } 860 }
OLDNEW
« no previous file with comments | « .gn ('k') | build/all.gyp » ('j') | third_party/openh264/BUILD.gn » ('J')

Powered by Google App Engine
This is Rietveld 408576698