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

Side by Side Diff: build/config/features.gni

Issue 1446453004: Adding third_party/openh264 build files for encoding (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Moved BUILD.gn deps to 'both_gn_and_gyp', added corresponding change to gn_migration.gypi 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
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 file contains Chrome-feature-related build flags (see ui.gni for 5 # This file contains Chrome-feature-related build flags (see ui.gni for
6 # UI-related ones). These should theoretically be moved to the build files of 6 # UI-related ones). These should theoretically be moved to the build files of
7 # the features themselves. 7 # the features themselves.
8 # 8 #
9 # However, today we have many "bad" dependencies on some of these flags from, 9 # However, today we have many "bad" dependencies on some of these flags from,
10 # e.g. base, so they need to be global to match the GYP configuration. Also, 10 # e.g. base, so they need to be global to match the GYP configuration. Also,
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 # If debug_devtools is set to true, JavaScript files for DevTools are stored 47 # If debug_devtools is set to true, JavaScript files for DevTools are stored
48 # as is and loaded from disk. Otherwise, a concatenated file is stored in 48 # as is and loaded from disk. Otherwise, a concatenated file is stored in
49 # resources.pak. It is still possible to load JS files from disk by passing 49 # resources.pak. It is still possible to load JS files from disk by passing
50 # --debug-devtools cmdline switch. 50 # --debug-devtools cmdline switch.
51 debug_devtools = false 51 debug_devtools = false
52 52
53 # Enables WebRTC. 53 # Enables WebRTC.
54 # TODO(GYP) make mac work. 54 # TODO(GYP) make mac work.
55 enable_webrtc = !is_ios && !is_mac 55 enable_webrtc = !is_ios && !is_mac
56 56
57 # Enable this to build OpenH264.
58 # CHECK THE OPENH264 LICENSE BEFORE BUILDING. The default value should be 0,
59 # the open source version of Chromium should not build this by default due to
60 # license and patent reasons.
61 use_openh264 = true
phoglund_chromium 2015/11/16 14:51:41 Same here.
phoglund_chromium 2015/11/16 14:52:21 Acknowledged.
Dirk Pranke 2015/11/17 02:54:19 "The default value should be false", not 0. Also,
Dirk Pranke 2015/11/17 23:49:14 One more comment ... we shouldn't be adding new se
hbos_chromium 2015/11/18 15:55:03 Done. Did the corresponding thing for gyp. + open
62
57 # Enables the Media Router. 63 # Enables the Media Router.
58 enable_media_router = !is_ios 64 enable_media_router = !is_ios
59 65
60 # Enables proprietary codecs and demuxers; e.g. H264, MOV, AAC, and MP3. 66 # Enables proprietary codecs and demuxers; e.g. H264, MOV, AAC, and MP3.
61 # Android OS includes support for proprietary codecs regardless of building 67 # Android OS includes support for proprietary codecs regardless of building
62 # Chromium or Google Chrome. We also ship Google Chrome and Chromecast with 68 # Chromium or Google Chrome. We also ship Google Chrome and Chromecast with
63 # proprietary codecs. 69 # proprietary codecs.
64 proprietary_codecs = is_android || is_chrome_branded || is_chromecast 70 proprietary_codecs = is_android || is_chrome_branded || is_chromecast
65 71
66 # TODO(GYP) This should be enabled on ios as well. 72 # TODO(GYP) This should be enabled on ios as well.
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
196 use_gconf = is_linux && !is_chromeos && !is_chromecast && !is_headless 202 use_gconf = is_linux && !is_chromeos && !is_chromecast && !is_headless
197 203
198 # Whether to back up data before sync. 204 # Whether to back up data before sync.
199 enable_pre_sync_backup = is_win || is_mac || (is_linux && !is_chromeos) 205 enable_pre_sync_backup = is_win || is_mac || (is_linux && !is_chromeos)
200 206
201 # Enable WebVR support by default on Android 207 # Enable WebVR support by default on Android
202 # Still requires command line flag to access API 208 # Still requires command line flag to access API
203 enable_webvr = is_android 209 enable_webvr = is_android
204 210
205 use_gio = is_desktop_linux && !is_headless 211 use_gio = is_desktop_linux && !is_headless
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698