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

Unified Diff: content/content_tests.gypi

Issue 14932020: Add Create() function to AudioCodecBridge and VideoCodecBridge to allow return of null pointers (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: addressing comments Created 7 years, 7 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: content/content_tests.gypi
diff --git a/content/content_tests.gypi b/content/content_tests.gypi
index 83902ca47fd01710c59e0aadb21196b617135099..30cdfc9c1f22deb6811c042e7d680591cac3fd1e 100644
--- a/content/content_tests.gypi
+++ b/content/content_tests.gypi
@@ -964,13 +964,11 @@
},
],
}],
- ['chromeos==1 or OS=="win"', {
- # TODO(felipeg): Make video_decode_accelerator_unittest work on Android.
- # http://crbug.com/178647
+ ['chromeos==1 or OS=="win" or OS=="android"', {
'targets': [
{
'target_name': 'video_decode_accelerator_unittest',
- 'type': 'executable',
+ 'type': '<(gtest_target_type)',
'dependencies': [
'content',
'../base/base.gyp:base',
@@ -983,11 +981,28 @@
'<(DEPTH)/third_party/khronos',
],
'sources': [
+ 'common/gpu/media/android_video_decode_accelerator_unittest.cc',
'common/gpu/media/rendering_helper.h',
'common/gpu/media/rendering_helper_gl.cc',
'common/gpu/media/video_decode_accelerator_unittest.cc',
],
'conditions': [
+ ['OS=="android"', {
+ 'sources/': [
+ ['exclude', '^common/gpu/media/rendering_helper.h'],
+ ['exclude', '^common/gpu/media/rendering_helper_gl.cc'],
+ ['exclude', '^common/gpu/media/video_decode_accelerator_unittest.cc'],
+ ],
+ 'dependencies': [
+ '../testing/gmock.gyp:gmock',
+ '../testing/android/native_test.gyp:native_test_native_code',
+ '../gpu/gpu.gyp:gpu_unittest_utils',
+ ],
+ }, { # OS!="android"
+ 'sources/': [
+ ['exclude', '^common/gpu/media/android_video_decode_accelerator_unittest.cc'],
+ ],
+ }],
['target_arch=="arm"', {
'include_dirs': [
'<(DEPTH)/third_party/openmax/il',
@@ -1077,6 +1092,18 @@
},
'includes': [ '../build/java_apk.gypi' ],
},
+ {
+ 'target_name': 'video_decode_accelerator_unittest_apk',
+ 'type': 'none',
+ 'dependencies': [
+ 'video_decode_accelerator_unittest',
+ ],
+ 'variables': {
+ 'test_suite_name': 'video_decode_accelerator_unittest',
+ 'input_shlib_path': '<(SHARED_LIB_DIR)/<(SHARED_LIB_PREFIX)content_unittests<(SHARED_LIB_SUFFIX)',
+ },
+ 'includes': [ '../build/apk_test.gypi' ],
+ },
],
}],
['OS == "android"', {

Powered by Google App Engine
This is Rietveld 408576698