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

Unified Diff: media/media_variables.gypi

Issue 1230793009: Initial support for the desktop media pipeline on Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed comments Created 5 years, 5 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
Index: media/media_variables.gypi
diff --git a/media/media_variables.gypi b/media/media_variables.gypi
new file mode 100644
index 0000000000000000000000000000000000000000..c27cc170318aa260c26f51a0c9e6a8c5c12533d2
--- /dev/null
+++ b/media/media_variables.gypi
@@ -0,0 +1,26 @@
+# Copyright 2015 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+# Common media variables.
+{
+ 'variables': {
+ 'conditions': [
+ ['OS == "android" or OS == "ios"', {
+ # Android and iOS don't use FFmpeg or libvpx by default.
xhwang 2015/07/15 00:14:20 What's gonna happen when I set build_ffmpegsumo=1
watk 2015/07/16 20:55:14 Your build won't work :) But even if we weren't se
+ # Set build_ffmpegsumo=1 for Android builds to compile experimental
+ # support for FFmpeg and the desktop media pipeline.
+ # This 'variables' dict is required to ensure build_ffmpegsumo is
+ # defined when we read its value below.
+ 'variables': {
+ 'build_ffmpegsumo%': 0,
+ },
+ 'media_use_ffmpeg%': '<(build_ffmpegsumo)',
xhwang 2015/07/15 00:14:20 I found "media_use_ffmpeg" more intuitive than "bu
+ 'media_use_libvpx%': 0,
+ }, {
+ 'media_use_ffmpeg%': 1,
+ 'media_use_libvpx%': 1,
+ }],
+ ],
+ },
+}

Powered by Google App Engine
This is Rietveld 408576698