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

Unified Diff: media/media.gyp

Issue 2031073002: Make media/capture gn and gyps produce components (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added again 'Android missed dep' PS5 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 afcb0d3a98c0f2087994fbc51f33d91246c9c327..ae921dfdda3fae804374791f490b61b4ba755d0b 100644
--- a/media/media.gyp
+++ b/media/media.gyp
@@ -49,7 +49,6 @@
],
},
'includes': [
- 'capture/capture.gypi',
'media_cdm.gypi',
'media_variables.gypi',
],
@@ -103,11 +102,7 @@
'include_dirs': [
'..',
],
- 'includes': [
- 'capture/capture.gypi',
- ],
'sources': [
- '<@(capture_sources)',
'audio/agc_audio_stream.h',
'audio/alsa/alsa_input.cc',
'audio/alsa/alsa_input.h',
@@ -476,8 +471,6 @@
'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',
@@ -714,11 +707,9 @@
}],
['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',
@@ -821,12 +812,6 @@
'audio/cras/cras_unified.h',
],
}],
-
- ['use_udev==1', {
- 'dependencies': [
- '<(DEPTH)/device/udev_linux/udev.gyp:udev_linux',
- ],
- }],
],
}],
['OS!="linux"', {
@@ -924,10 +909,6 @@
],
}],
['OS=="mac"', {
- 'dependencies': [
- '<(DEPTH)/third_party/decklink/decklink.gyp:decklink',
- ],
-
'link_settings': {
'libraries': [
'$(SDKROOT)/System/Library/Frameworks/AudioToolbox.framework',
@@ -940,39 +921,6 @@
},
}],
['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': [
@@ -1128,6 +1076,8 @@
'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',
@@ -1150,7 +1100,6 @@
'../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',
@@ -1546,12 +1495,14 @@
},
{
# GN version: //media:audio_unittests
- # For running the subset of media_unittests that might require audio
+ # For running the subset of tests 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',
@@ -1749,6 +1700,56 @@
},
], # 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,7 +1757,6 @@
'target_name': 'media_unittests_apk',
'type': 'none',
'dependencies': [
- 'capture_java',
'media_java',
'media_unittests',
],
@@ -1771,7 +1771,6 @@
'target_name': 'media_perftests_apk',
'type': 'none',
'dependencies': [
- 'capture_java',
'media_java',
'media_perftests',
],
@@ -1800,19 +1799,6 @@
'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',
@@ -1902,23 +1888,6 @@
],
},
{
- # 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',
@@ -1933,24 +1902,6 @@
},
'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