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

Side by Side Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py

Issue 2143123004: Access environment variables through Host object instead of directly. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased and updated after change for dummy home dir on linux. Created 4 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 unified diff | Download patch
OLDNEW
1 # Copyright (C) 2010 Google Inc. All rights reserved. 1 # Copyright (C) 2010 Google Inc. All rights reserved.
2 # Copyright (C) 2010 Gabor Rapcsanyi (rgabor@inf.u-szeged.hu), University of Sze ged 2 # Copyright (C) 2010 Gabor Rapcsanyi (rgabor@inf.u-szeged.hu), University of Sze ged
3 # Copyright (C) 2011 Apple Inc. All rights reserved. 3 # Copyright (C) 2011 Apple Inc. All rights reserved.
4 # 4 #
5 # Redistribution and use in source and binary forms, with or without 5 # Redistribution and use in source and binary forms, with or without
6 # modification, are permitted provided that the following conditions are 6 # modification, are permitted provided that the following conditions are
7 # met: 7 # met:
8 # 8 #
9 # * Redistributions of source code must retain the above copyright 9 # * Redistributions of source code must retain the above copyright
10 # notice, this list of conditions and the following disclaimer. 10 # notice, this list of conditions and the following disclaimer.
(...skipping 455 matching lines...) Expand 10 before | Expand all | Expand 10 after
466 466
467 return option_parser.parse_args(args) 467 return option_parser.parse_args(args)
468 468
469 469
470 def _set_up_derived_options(port, options, args): 470 def _set_up_derived_options(port, options, args):
471 """Sets the options values that depend on other options values.""" 471 """Sets the options values that depend on other options values."""
472 if options.batch_size is None: 472 if options.batch_size is None:
473 options.batch_size = port.default_batch_size() 473 options.batch_size = port.default_batch_size()
474 474
475 if not options.child_processes: 475 if not options.child_processes:
476 options.child_processes = os.environ.get("WEBKIT_TEST_CHILD_PROCESSES", 476 options.child_processes = port.host.environ.get("WEBKIT_TEST_CHILD_PROCE SSES",
477 str(port.default_child_processe s())) 477 str(port.default_child_p rocesses()))
478 if not options.max_locked_shards: 478 if not options.max_locked_shards:
479 options.max_locked_shards = int(os.environ.get("WEBKIT_TEST_MAX_LOCKED_S HARDS", 479 options.max_locked_shards = int(port.host.environ.get("WEBKIT_TEST_MAX_L OCKED_SHARDS",
480 str(port.default_max_lock ed_shards()))) 480 str(port.default_m ax_locked_shards())))
481 481
482 if not options.configuration: 482 if not options.configuration:
483 options.configuration = port.default_configuration() 483 options.configuration = port.default_configuration()
484 484
485 if options.pixel_tests is None: 485 if options.pixel_tests is None:
486 options.pixel_tests = port.default_pixel_tests() 486 options.pixel_tests = port.default_pixel_tests()
487 487
488 if not options.time_out_ms: 488 if not options.time_out_ms:
489 options.time_out_ms = str(port.default_timeout_ms()) 489 options.time_out_ms = str(port.default_timeout_ms())
490 490
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
576 _log.debug("\t%s" % process) 576 _log.debug("\t%s" % process)
577 577
578 return run_details 578 return run_details
579 579
580 finally: 580 finally:
581 printer.cleanup() 581 printer.cleanup()
582 582
583 if __name__ == '__main__': 583 if __name__ == '__main__':
584 exit_code = main(sys.argv[1:], sys.stdout, sys.stderr) 584 exit_code = main(sys.argv[1:], sys.stdout, sys.stderr)
585 sys.exit(exit_code) 585 sys.exit(exit_code)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698