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

Side by Side Diff: media/media.gyp

Issue 1727953005: Enable AudioDecoder unit tests on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Compile audio_decoder_unittest.cc only if media_use_ffmpeg is set Created 4 years, 9 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
« media/base/BUILD.gn ('K') | « media/filters/audio_decoder_unittest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 { 5 {
6 'variables': { 6 'variables': {
7 'chromium_code': 1, 7 'chromium_code': 1,
8 # Override to dynamically link the cras (ChromeOS audio) library. 8 # Override to dynamically link the cras (ChromeOS audio) library.
9 'use_cras%': 0, 9 'use_cras%': 0,
10 # Option e.g. for Linux distributions to link pulseaudio directly 10 # Option e.g. for Linux distributions to link pulseaudio directly
(...skipping 1359 matching lines...) Expand 10 before | Expand all | Expand 10 after
1370 'sources': [ 1370 'sources': [
1371 'filters/h265_parser_unittest.cc', 1371 'filters/h265_parser_unittest.cc',
1372 ], 1372 ],
1373 }], 1373 }],
1374 ['media_use_ffmpeg==1', { 1374 ['media_use_ffmpeg==1', {
1375 'dependencies': [ 1375 'dependencies': [
1376 '../third_party/ffmpeg/ffmpeg.gyp:ffmpeg', 1376 '../third_party/ffmpeg/ffmpeg.gyp:ffmpeg',
1377 ], 1377 ],
1378 }, { # media_use_ffmpeg==0 1378 }, { # media_use_ffmpeg==0
1379 'sources!': [ 1379 'sources!': [
1380 'base/container_names_unittest.cc',
DaleCurtis 2016/02/29 23:17:46 Same Q, why?
1380 'ffmpeg/ffmpeg_common_unittest.cc', 1381 'ffmpeg/ffmpeg_common_unittest.cc',
1381 'filters/audio_decoder_unittest.cc', 1382 'filters/audio_decoder_unittest.cc',
1382 'filters/ffmpeg_aac_bitstream_converter_unittest.cc', 1383 'filters/ffmpeg_aac_bitstream_converter_unittest.cc',
1383 'filters/ffmpeg_demuxer_unittest.cc', 1384 'filters/ffmpeg_demuxer_unittest.cc',
1384 'filters/ffmpeg_glue_unittest.cc', 1385 'filters/ffmpeg_glue_unittest.cc',
1385 'filters/ffmpeg_h264_to_annex_b_bitstream_converter_unittest.cc', 1386 'filters/ffmpeg_h264_to_annex_b_bitstream_converter_unittest.cc',
1386 ], 1387 ],
1387 }], 1388 }],
1388 # Even if FFmpeg is enabled on Android we don't want these. 1389 # Even if FFmpeg is enabled on Android we don't want these.
1389 # TODO(watk): Refactor tests that could be made to run on Android. See 1390 # TODO(watk): Refactor tests that could be made to run on Android. See
1390 # http://crbug.com/570762 1391 # http://crbug.com/570762
1391 ['media_use_ffmpeg==0 or OS=="android"', { 1392 ['media_use_ffmpeg==0 or OS=="android"', {
1392 'sources!': [ 1393 'sources!': [
1393 'base/audio_video_metadata_extractor_unittest.cc', 1394 'base/audio_video_metadata_extractor_unittest.cc',
1394 'base/media_file_checker_unittest.cc', 1395 'base/media_file_checker_unittest.cc',
1395 'filters/ffmpeg_video_decoder_unittest.cc', 1396 'filters/ffmpeg_video_decoder_unittest.cc',
1396 'test/pipeline_integration_test.cc', 1397 'test/pipeline_integration_test.cc',
1397 'test/pipeline_integration_test_base.cc', 1398 'test/pipeline_integration_test_base.cc',
1398 1399
1399 # These tests are confused by Android always having proprietary 1400 # These tests are confused by Android always having proprietary
1400 # codecs enabled, but ffmpeg_branding=Chromium. These should be 1401 # codecs enabled, but ffmpeg_branding=Chromium. These should be
1401 # fixed, see http://crbug.com/570762. 1402 # fixed, see http://crbug.com/570762.
1402 'filters/audio_decoder_unittest.cc',
1403 'filters/audio_file_reader_unittest.cc', 1403 'filters/audio_file_reader_unittest.cc',
1404 'filters/ffmpeg_demuxer_unittest.cc', 1404 'filters/ffmpeg_demuxer_unittest.cc',
1405 ], 1405 ],
1406 }], 1406 }],
1407 ['OS=="android"', { 1407 ['OS=="android"', {
1408 'dependencies': [ 1408 'dependencies': [
1409 '../testing/android/native_test.gyp:native_test_native_code', 1409 '../testing/android/native_test.gyp:native_test_native_code',
1410 'player_android', 1410 'player_android',
1411 ], 1411 ],
1412 'sources!': [ 1412 'sources!': [
(...skipping 766 matching lines...) Expand 10 before | Expand all | Expand 10 after
2179 'dependencies': [ 2179 'dependencies': [
2180 '../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck', 2180 '../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
2181 ], 2181 ],
2182 }], 2182 }],
2183 ], 2183 ],
2184 }, 2184 },
2185 ], 2185 ],
2186 }], 2186 }],
2187 ], 2187 ],
2188 } 2188 }
OLDNEW
« media/base/BUILD.gn ('K') | « media/filters/audio_decoder_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698