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

Unified Diff: media/media.gyp

Issue 2083813003: Revert of Re-Reland: Make media/capture gn and gyps produce components (patchset #2 id:20001 of htt… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2774
Patch Set: Created 4 years, 6 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 | « media/gpu/BUILD.gn ('k') | media/media_gpu.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/media.gyp
diff --git a/media/media.gyp b/media/media.gyp
index 6bfddabc25404693805537d5c21ae16561576e18..a553800a0d2b1ca0d6a74f2e6fbb5dd195ba4585 100644
--- a/media/media.gyp
+++ b/media/media.gyp
@@ -49,6 +49,7 @@
],
},
'includes': [
+ 'capture/capture.gypi',
'media_cdm.gypi',
'media_variables.gypi',
],
@@ -101,7 +102,11 @@
'include_dirs': [
'..',
],
+ 'includes': [
+ 'capture/capture.gypi',
+ ],
'sources': [
+ '<@(capture_sources)',
'audio/agc_audio_stream.h',
'audio/alsa/alsa_input.cc',
'audio/alsa/alsa_input.h',
@@ -470,6 +475,8 @@
'base/video_util.h',
'base/wall_clock_time_source.cc',
'base/wall_clock_time_source.h',
+ 'base/win/mf_initializer.cc',
+ 'base/win/mf_initializer.h',
'base/yuv_convert.cc',
'base/yuv_convert.h',
'cdm/aes_decryptor.cc',
@@ -706,9 +713,11 @@
}],
['OS=="android"', {
'dependencies': [
+ 'capture_java',
'media_android_jni_headers',
'media_java',
'player_android',
+ 'video_capture_android_jni_headers',
],
'sources!': [
'base/audio_video_metadata_extractor.cc',
@@ -811,6 +820,12 @@
'audio/cras/cras_unified.h',
],
}],
+
+ ['use_udev==1', {
+ 'dependencies': [
+ '<(DEPTH)/device/udev_linux/udev.gyp:udev_linux',
+ ],
+ }],
],
}],
['OS!="linux"', {
@@ -908,6 +923,10 @@
],
}],
['OS=="mac"', {
+ 'dependencies': [
+ '<(DEPTH)/third_party/decklink/decklink.gyp:decklink',
+ ],
+
'link_settings': {
'libraries': [
'$(SDKROOT)/System/Library/Frameworks/AudioToolbox.framework',
@@ -920,6 +939,39 @@
},
}],
['OS=="win"', {
+ 'link_settings': {
+ 'libraries': [
+ '-ldxguid.lib',
+ '-lmf.lib',
+ '-lmfplat.lib',
+ '-lmfreadwrite.lib',
+ '-lmfuuid.lib',
+ '-lsetupapi.lib',
+ '-lwinmm.lib',
+ ],
+ },
+ # Specify delayload for media.dll.
+ 'msvs_settings': {
+ 'VCLinkerTool': {
+ 'DelayLoadDLLs': [
+ 'mf.dll',
+ 'mfplat.dll',
+ 'mfreadwrite.dll',
+ ],
+ },
+ },
+ # Specify delayload for components that link with media.lib.
+ 'all_dependent_settings': {
+ 'msvs_settings': {
+ 'VCLinkerTool': {
+ 'DelayLoadDLLs': [
+ 'mf.dll',
+ 'mfplat.dll',
+ 'mfreadwrite.dll',
+ ],
+ },
+ },
+ },
# TODO(wolenetz): Fix size_t to int truncations in win64. See
# http://crbug.com/171009
'conditions': [
@@ -1075,8 +1127,6 @@
'type': '<(gtest_target_type)',
'dependencies': [
'audio_test_config',
- # TODO(mcasas): Remove this entry after https://crbug.com/618718.
- 'capture/capture.gyp:unittests',
'cdm_paths',
'media',
'media_features',
@@ -1099,6 +1149,7 @@
'../url/url.gyp:url_lib',
],
'sources': [
+ '<@(capture_unittests_sources)',
'base/android/access_unit_queue_unittest.cc',
'base/android/media_codec_decoder_unittest.cc',
'base/android/media_drm_bridge_unittest.cc',
@@ -1494,14 +1545,12 @@
},
{
# GN version: //media:audio_unittests
- # For running the subset of tests that might require audio
+ # For running the subset of media_unittests that might require audio
# hardware separately on GPU bots. media_unittests includes these too.
'target_name': 'audio_unittests',
'type': '<(gtest_target_type)',
'dependencies': [
'audio_test_config',
- # TODO(mcasas): Remove this entry after https://crbug.com/618718.
- 'capture/capture.gyp:unittests',
'media_test_support',
'../base/base.gyp:test_support_base',
'../testing/gmock.gyp:gmock',
@@ -1699,56 +1748,6 @@
},
], # targets
}],
- ['OS=="win"', {
- 'targets': [
- {
- # GN version: //media/base/win
- 'target_name': 'mf_initializer',
- 'type': '<(component)',
- 'include_dirs': [ '..', ],
- 'defines': [ 'MF_INITIALIZER_IMPLEMENTATION', ],
- 'sources': [
- 'base/win/mf_initializer_export.h',
- 'base/win/mf_initializer.cc',
- 'base/win/mf_initializer.h',
- ],
- 'dependencies': [
- '../base/base.gyp:base',
- ],
- 'link_settings': {
- 'libraries': [
- '-ldxguid.lib',
- '-lmf.lib',
- '-lmfplat.lib',
- '-lmfreadwrite.lib',
- '-lmfuuid.lib',
- '-lsetupapi.lib',
- '-lwinmm.lib',
- ],
- },
- 'msvs_settings': {
- 'VCLinkerTool': {
- 'DelayLoadDLLs': [
- 'mf.dll',
- 'mfplat.dll',
- 'mfreadwrite.dll',
- ],
- },
- },
- 'all_dependent_settings': {
- 'msvs_settings': {
- 'VCLinkerTool': {
- 'DelayLoadDLLs': [
- 'mf.dll',
- 'mfplat.dll',
- 'mfreadwrite.dll',
- ],
- },
- },
- },
- },
- ],
- }],
['OS=="android"', {
'targets': [
{
@@ -1756,6 +1755,7 @@
'target_name': 'media_unittests_apk',
'type': 'none',
'dependencies': [
+ 'capture_java',
'media_java',
'media_unittests',
],
@@ -1770,6 +1770,7 @@
'target_name': 'media_perftests_apk',
'type': 'none',
'dependencies': [
+ 'capture_java',
'media_java',
'media_perftests',
],
@@ -1798,6 +1799,19 @@
'includes': ['../build/jni_generator.gypi'],
},
{
+ # GN: //media/capture/video/android:capture_jni_headers
+ 'target_name': 'video_capture_android_jni_headers',
+ 'type': 'none',
+ 'sources': [
+ 'capture/video/android/java/src/org/chromium/media/VideoCapture.java',
+ 'capture/video/android/java/src/org/chromium/media/VideoCaptureFactory.java',
+ ],
+ 'variables': {
+ 'jni_gen_package': 'media',
+ },
+ 'includes': ['../build/jni_generator.gypi'],
+ },
+ {
# GN: //media/base/android:android
'target_name': 'player_android',
'type': 'static_library',
@@ -1887,6 +1901,23 @@
],
},
{
+ # GN: //media/capture/video/android:capture_java
+ 'target_name': 'capture_java',
+ 'type': 'none',
+ 'dependencies': [
+ '../base/base.gyp:base',
+ 'media_android_captureapitype',
+ 'media_android_imageformat',
+ ],
+ 'export_dependent_settings': [
+ '../base/base.gyp:base',
+ ],
+ 'variables': {
+ 'java_in_dir': 'capture/video/android/java',
+ },
+ 'includes': ['../build/java.gypi'],
+ },
+ {
# GN: //media/base/android:media_java
'target_name': 'media_java',
'type': 'none',
@@ -1901,6 +1932,24 @@
},
'includes': ['../build/java.gypi'],
},
+ {
+ # GN: //media/base/android:media_android_captureapitype
+ 'target_name': 'media_android_captureapitype',
+ 'type': 'none',
+ 'variables': {
+ 'source_file': 'capture/video/video_capture_device.h',
+ },
+ 'includes': [ '../build/android/java_cpp_enum.gypi' ],
+ },
+ {
+ # GN: //media/base/android:media_android_imageformat
+ 'target_name': 'media_android_imageformat',
+ 'type': 'none',
+ 'variables': {
+ 'source_file': 'capture/video/android/video_capture_device_android.h',
+ },
+ 'includes': [ '../build/android/java_cpp_enum.gypi' ],
+ },
],
'conditions': [
['test_isolation_mode != "noop"',
« no previous file with comments | « media/gpu/BUILD.gn ('k') | media/media_gpu.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698