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

Side by Side Diff: media/BUILD.gn

Issue 1727953005: Enable AudioDecoder unit tests on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Undid the split, disabled ffmpeg tests on Android Created 4 years, 10 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 unified diff | Download patch
« no previous file with comments | « no previous file | media/base/audio_hash.h » ('j') | media/base/audio_hash.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 import("//build/buildflag_header.gni") 5 import("//build/buildflag_header.gni")
6 import("//build/config/android/config.gni") 6 import("//build/config/android/config.gni")
7 import("//build/config/arm.gni") 7 import("//build/config/arm.gni")
8 import("//build/config/features.gni") 8 import("//build/config/features.gni")
9 import("//build/config/linux/pkg_config.gni") 9 import("//build/config/linux/pkg_config.gni")
10 import("//build/config/ui.gni") 10 import("//build/config/ui.gni")
(...skipping 717 matching lines...) Expand 10 before | Expand all | Expand 10 after
728 "//ui/android:ui_java", 728 "//ui/android:ui_java",
729 ] 729 ]
730 } 730 }
731 731
732 # If ExternalClearKey is built, we can test CdmAdapter. 732 # If ExternalClearKey is built, we can test CdmAdapter.
733 if (enable_pepper_cdms) { 733 if (enable_pepper_cdms) {
734 sources += [ "cdm/cdm_adapter_unittest.cc" ] 734 sources += [ "cdm/cdm_adapter_unittest.cc" ]
735 data_deps += [ "//media/cdm/ppapi:clearkeycdm" ] 735 data_deps += [ "//media/cdm/ppapi:clearkeycdm" ]
736 } 736 }
737 737
738 if (media_use_ffmpeg || is_android) {
DaleCurtis 2016/02/26 23:31:42 Just add the file after l.751
Tima Vaisburd 2016/02/29 23:07:14 Done. This makes audio_decoder_unittest available
739 sources += [
740 # This file disables FFmpeg decoder tests on Android.
741 # FFmpeg tests should be eventially reenabled, http://crbug.com/570762.
742 "filters/audio_decoder_unittest.cc",
743 ]
744 }
745
738 if (media_use_ffmpeg) { 746 if (media_use_ffmpeg) {
739 sources += [ 747 sources += [
740 "ffmpeg/ffmpeg_common_unittest.cc", 748 "ffmpeg/ffmpeg_common_unittest.cc",
741 "filters/blocking_url_protocol_unittest.cc", 749 "filters/blocking_url_protocol_unittest.cc",
742 "filters/ffmpeg_glue_unittest.cc", 750 "filters/ffmpeg_glue_unittest.cc",
743 "filters/in_memory_url_protocol_unittest.cc", 751 "filters/in_memory_url_protocol_unittest.cc",
744 ] 752 ]
745 753
746 if (!is_android) { 754 if (!is_android) {
747 sources += [ 755 sources += [
748 # These tests are confused by Android always having proprietary 756 # These tests are confused by Android always having proprietary
749 # codecs enabled, but ffmpeg_branding=Chromium. These should be 757 # codecs enabled, but ffmpeg_branding=Chromium. These should be
750 # fixed, http://crbug.com/570762. 758 # fixed, http://crbug.com/570762.
751 "filters/audio_decoder_unittest.cc",
752 "filters/audio_file_reader_unittest.cc", 759 "filters/audio_file_reader_unittest.cc",
753 "filters/ffmpeg_demuxer_unittest.cc", 760 "filters/ffmpeg_demuxer_unittest.cc",
754 761
755 # FFmpeg on Android does not include video decoders. 762 # FFmpeg on Android does not include video decoders.
756 "filters/ffmpeg_video_decoder_unittest.cc", 763 "filters/ffmpeg_video_decoder_unittest.cc",
757 ] 764 ]
758 } 765 }
759 } 766 }
760 767
761 if (current_cpu != "arm" && is_chromeos) { 768 if (current_cpu != "arm" && is_chromeos) {
(...skipping 176 matching lines...) Expand 10 before | Expand all | Expand 10 after
938 if (proprietary_codecs) { 945 if (proprietary_codecs) {
939 fuzzer_test("media_cenc_utils_fuzzer") { 946 fuzzer_test("media_cenc_utils_fuzzer") {
940 sources = [ 947 sources = [
941 "cdm/cenc_utils_fuzzertest.cc", 948 "cdm/cenc_utils_fuzzertest.cc",
942 ] 949 ]
943 deps = [ 950 deps = [
944 ":media", 951 ":media",
945 ] 952 ]
946 } 953 }
947 } 954 }
OLDNEW
« no previous file with comments | « no previous file | media/base/audio_hash.h » ('j') | media/base/audio_hash.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698