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

Unified Diff: chrome/test/functional/media/media_test_runner.py

Issue 6992026: Bug fixes in matrix driven test and some minor changes in perf media test. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 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
« no previous file with comments | « chrome/test/functional/media/media_test_matrix_unittest.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/functional/media/media_test_runner.py
diff --git a/chrome/test/functional/media/media_test_runner.py b/chrome/test/functional/media/media_test_runner.py
index 58c7d443f2eed55a457b55be7d960455a8f02d8b..e45c6d9df3e326a39da090346a8cacb96c8bf756 100755
--- a/chrome/test/functional/media/media_test_runner.py
+++ b/chrome/test/functional/media/media_test_runner.py
@@ -93,16 +93,18 @@ def main():
default=DEFAULT_MEASURE_INTERVALS,
help='Interval for measurement data [defaults to "%d"]' %
DEFAULT_MEASURE_INTERVALS)
- parser.add_option('-o', '--test-one-combination', dest='one_combination',
- default=True, # Currently default is True
- # since we want to test only 1 combination.
- help='Run only one parameter combination')
+ parser.add_option('-c', '--cache_test', dest='cache_test',
+ default=False, help='Include cache test',
+ action='store_true')
+ parser.add_option('-z', '--test-one-video', dest='one_video',
+ default=False, help='Run only one video',
+ action='store_true')
parser.add_option(
'-w', '--test_scenario_input_filename',
dest='test_scenario_input_filename',
default='', help='Test scenario file (CSV form)', metavar='FILE')
parser.add_option(
- '-c', '--test_scenario', dest='test_scenario',
+ '-q', '--test_scenario', dest='test_scenario',
default='', help='Test scenario (action triples delimited by \'|\')')
parser.add_option('-s', '--suite', dest='suite',
help='Suite file')
@@ -174,6 +176,8 @@ def main():
parent_envs = copy.deepcopy(os.environ)
if options.input_matrix_filename is None:
par_filename = os.path.join(os.pardir, filename)
+ else:
+ par_filename = filename
envs = {
MediaTestEnvNames.MEDIA_TAG_ENV_NAME: tag,
MediaTestEnvNames.MEDIA_FILENAME_ENV_NAME: par_filename,
@@ -227,8 +231,10 @@ def main():
cmd += ' -v'
proc = Popen(cmd, env=envs, shell=True)
proc.communicate()
- if options.one_combination:
- sys.exit(0)
+ if not options.cache_test:
+ break
+ if options.one_video:
+ break
if __name__ == '__main__':
« no previous file with comments | « chrome/test/functional/media/media_test_matrix_unittest.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698