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

Unified Diff: chrome/browser_tests.isolate

Issue 1063453004: Copy Widevine files for swarming tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove extra quote Created 5 years, 8 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
« build/isolate.gypi ('K') | « build/isolate.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser_tests.isolate
diff --git a/chrome/browser_tests.isolate b/chrome/browser_tests.isolate
index 7d97d28f4b309eeb8c650dfe894d9f4e509b4525..dba705aac210b9e9531f68253ad2d4b77c7af900 100644
--- a/chrome/browser_tests.isolate
+++ b/chrome/browser_tests.isolate
@@ -41,6 +41,14 @@
],
},
}],
+ ['OS=="linux" and branding == "Chrome" and enable_pepper_cdms == 1', {
Nico 2015/04/07 20:36:33 no spaces around ==
M-A Ruel 2015/04/07 20:57:51 Why branding=="Chrome" is needed?
Nico 2015/04/07 21:01:36 Because libwidewinecdm.so only exists in that case
jrummell 2015/04/07 21:06:01 Widevine is only available with Chrome, and not Ch
M-A Ruel 2015/04/07 21:57:30 As Nico said, it would have been great if there wa
jrummell 2015/04/07 22:01:41 Done.
+ 'variables': {
+ 'files': [
+ '<(PRODUCT_DIR)/libwidevinecdm.so',
+ '<(PRODUCT_DIR)/libwidevinecdmadapter.so',
+ ],
+ },
+ }],
['OS=="linux" and libpeer_target_type=="loadable_module"', {
'variables': {
'files': [
@@ -182,6 +190,14 @@
],
},
}],
+ ['OS=="mac" and branding == "Chrome" and enable_pepper_cdms == 1', {
Nico 2015/04/07 20:36:33 likewise
+ 'variables': {
+ 'files': [
+ '<(PRODUCT_DIR)/libwidevinecdm.dylib',
+ '<(PRODUCT_DIR)/widevinecdmadapter.plugin/',
+ ],
+ },
+ }],
['OS=="win"', {
'variables': {
'files': [
@@ -195,6 +211,14 @@
],
},
}],
+ ['OS=="win" and branding == "Chrome" and enable_pepper_cdms == 1', {
Nico 2015/04/07 20:36:33 likewise
+ 'variables': {
+ 'files': [
+ '<(PRODUCT_DIR)/widevinecdm.dll',
+ '<(PRODUCT_DIR)/widevinecdmadapter.dll',
+ ],
+ },
+ }],
['OS=="win" and target_arch=="ia32"', {
'variables': {
'files': [
« build/isolate.gypi ('K') | « build/isolate.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698