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

Unified Diff: media/BUILD.gn

Issue 2848213003: Fix building FFMPEG into Android (Closed)
Patch Set: Make decrypting decoder non-Android again Created 3 years, 8 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 | « content/browser/media/media_internals.cc ('k') | media/filters/decoder_selector.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/BUILD.gn
diff --git a/media/BUILD.gn b/media/BUILD.gn
index 8d97ec513ba2e43f871c4a10b1db5d90b40fb9c7..1e5cebcefd2194cb884b27491ad9db58ca99a15b 100644
--- a/media/BUILD.gn
+++ b/media/BUILD.gn
@@ -376,17 +376,12 @@ component("media") {
deps += [ "//third_party/libvpx" ]
}
- if (disable_ffmpeg_video_decoders) {
- sources -= [
- "filters/decrypting_video_decoder.cc",
- "filters/decrypting_video_decoder.h",
- ]
- }
-
if (is_android) {
sources -= [
"filters/decrypting_audio_decoder.cc",
"filters/decrypting_audio_decoder.h",
+ "filters/decrypting_video_decoder.cc",
+ "filters/decrypting_video_decoder.h",
]
sources += [
"filters/android/media_codec_audio_decoder.cc",
@@ -625,7 +620,7 @@ source_set("unit_tests") {
data_deps = []
- if (disable_ffmpeg_video_decoders) {
+ if (is_android) {
sources -= [
"filters/decrypting_audio_decoder_unittest.cc",
"filters/decrypting_video_decoder_unittest.cc",
@@ -659,9 +654,8 @@ source_set("unit_tests") {
]
if (!disable_ffmpeg_video_decoders) {
- sources += [
- "filters/ffmpeg_video_decoder_unittest.cc",
- ]
+ # FFmpeg on Android does not include video decoders.
+ sources += [ "filters/ffmpeg_video_decoder_unittest.cc" ]
}
}
« no previous file with comments | « content/browser/media/media_internals.cc ('k') | media/filters/decoder_selector.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698