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

Unified Diff: media/media_options.gni

Issue 1345013002: Add is_chromecast to GN build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Rebase + ui changes. Created 5 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/BUILD.gn ('k') | media/midi/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/media_options.gni
diff --git a/media/media_options.gni b/media/media_options.gni
index 746d96d6b9d7d2dd9edc9973b99549aa1befd98b..1f3f64438e1030b626d97863070d99f579d28a9b 100644
--- a/media/media_options.gni
+++ b/media/media_options.gni
@@ -2,6 +2,8 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
+import("//build/config/chromecast_build.gni")
+
declare_args() {
# Allows distributions to link pulseaudio directly (DT_NEEDED) instead of
# using dlopen. This helps with automated detection of ABI mismatches and
@@ -36,22 +38,27 @@ declare_args() {
# Enables runtime selection of ALSA library for audio.
use_alsa = false
- # TODO(GYP): How to handled the "embedded" use case?
- # Original conditional: (OS=="linux" or OS=="freebsd" or OS=="solaris") and embedded!=1
- if (is_posix && !is_android && !is_mac) {
+ # Alsa should be used on non-Android, non-Mac POSIX systems, and Chromecast
+ # builds for desktop Linux.
+ if ((is_posix && !is_android && !is_mac) ||
+ (is_chromecast && target_cpu != "arm" && is_linux)) {
use_alsa = true
if (!use_cras) {
use_pulseaudio = true
}
}
+ # Use low-memory buffers on non-Android builds of Chromecast.
+ use_low_memory_buffer = is_chromecast && !is_android
+
# Enables the MPEG2-TS stream parser for use with Media Source. Disabled by
- # default since it's not available on the normal Web Platform and costs money.
- enable_mpeg2ts_stream_parser = false
+ # default (except on Chromecast) since it's not available on the normal Web
+ # Platform and costs money.
+ enable_mpeg2ts_stream_parser = is_chromecast
# Enable HEVC/H265 demuxing. Actual decoding must be provided by the
- # platform.
- enable_hevc_demuxing = false
+ # platform. Enable by default for Chromecast.
+ enable_hevc_demuxing = is_chromecast
# Experiment to enable mojo media application: http://crbug.com/431776
# Valid options are:
« no previous file with comments | « media/BUILD.gn ('k') | media/midi/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698