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

Unified Diff: third_party/widevine/cdm/widevine_cdm.gyp

Issue 1957643002: media: Move widevine CDM targets to WidevineCdm folder (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: comments addressed Created 4 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: third_party/widevine/cdm/widevine_cdm.gyp
diff --git a/third_party/widevine/cdm/widevine_cdm.gyp b/third_party/widevine/cdm/widevine_cdm.gyp
index 97a416518bfb4063443695365f665d6870fa5acd..8b5bc7f2c09a83eb01ef029a856a768de75782b8 100644
--- a/third_party/widevine/cdm/widevine_cdm.gyp
+++ b/third_party/widevine/cdm/widevine_cdm.gyp
@@ -10,6 +10,7 @@
},
'enable_widevine%': '<(enable_widevine)',
'widevine_cdm_version_h_file%': 'widevine_cdm_version.h',
+ # TODO(xhwang): Also include manifest.json.
'widevine_cdm_binary_files%': [],
'conditions': [
[ 'branding == "Chrome"', {
@@ -86,6 +87,7 @@
{
# GN version: //third_party/widevine/cdm:widevinecdmadapter
'target_name': 'widevinecdmadapter',
+ 'product_dir': '<(PRODUCT_DIR)/WidevineCdm',
'type': 'none',
'conditions': [
[ '(branding == "Chrome" or enable_widevine == 1) and enable_pepper_cdms == 1', {
@@ -104,19 +106,19 @@
'libraries': [
'-lrt',
# Copied/created by widevinecdm.
- '<(PRODUCT_DIR)/libwidevinecdm.so',
+ '<(PRODUCT_DIR)/WidevineCdm/libwidevinecdm.so',
],
}],
[ 'OS == "win"', {
'libraries': [
# Copied/created by widevinecdm.
- '<(PRODUCT_DIR)/widevinecdm.dll.lib',
+ '<(PRODUCT_DIR)/WidevineCdm/widevinecdm.dll.lib',
],
}],
[ 'OS == "mac"', {
'libraries': [
# Copied/created by widevinecdm.
- '<(PRODUCT_DIR)/libwidevinecdm.dylib',
+ '<(PRODUCT_DIR)/WidevineCdm/libwidevinecdm.dylib',
],
}],
],
@@ -146,9 +148,7 @@
}],
],
'copies': [{
- # TODO(ddorwin): Do we need a sub-directory? We either need a
- # sub-directory or to rename manifest.json before we can copy it.
- 'destination': '<(PRODUCT_DIR)',
+ 'destination': '<(PRODUCT_DIR)/WidevineCdm',
'files': [ '<@(widevine_cdm_binary_files)' ],
}],
}],

Powered by Google App Engine
This is Rietveld 408576698