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

Unified Diff: testing/chromoting/integration_tests.gyp

Issue 1231663003: Add functionality to download files from GCS, and include them in the Chromoting isolate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 5 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: testing/chromoting/integration_tests.gyp
diff --git a/testing/chromoting/integration_tests.gyp b/testing/chromoting/integration_tests.gyp
index 40c80eeed89d5295509cd905b9b420c859b8f693..a7d733263c8232832329d7728b97c2e59280dbe9 100644
--- a/testing/chromoting/integration_tests.gyp
+++ b/testing/chromoting/integration_tests.gyp
@@ -47,6 +47,31 @@
'sources': [
'chromoting_integration_tests.isolate',
],
+ 'actions': [
+ {
+ 'action_name': 'download_test_files',
+ 'variables': {
+ 'dl_files_script': './download_test_files.py',
+ 'input_files': './test_files.txt',
joedow 2015/07/13 19:10:12 Does it make sense to have a single file which has
anandc 2015/07/14 01:03:07 Good suggestion. I tried listing the files as a va
+ 'output_folder': './',
+ },
+ 'inputs': [
+ '<(input_files)',
+ ],
+ 'outputs': [
+ '<(output_folder)',
+ ],
+ 'action': [
+ 'python',
+ '<(dl_files_script)',
+ '--input_files',
+ '<(input_files)',
+ '--output_folder',
+ '<(output_folder)',
+ ],
+ 'message': 'Downloading required Remoting test files.',
joedow 2015/07/13 19:10:12 I think message should say "Downloading required c
anandc 2015/07/14 01:03:07 Done.
+ },
+ ],
}, # target_name: 'chromoting_integration_tests_run'
],
}],

Powered by Google App Engine
This is Rietveld 408576698