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

Unified Diff: tools/testrunner/local/commands.py

Issue 2811103002: Revert of [date] Add ICU backend for timezone info behind a flag (Closed)
Patch Set: Created 3 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
« no previous file with comments | « test/mjsunit/testcfg.py ('k') | tools/testrunner/local/execution.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/testrunner/local/commands.py
diff --git a/tools/testrunner/local/commands.py b/tools/testrunner/local/commands.py
index b2dc74e4d4908376040cf6d6e3e68add942ce59d..94b892c34e1997d31f2529a136c1202fbb66658c 100644
--- a/tools/testrunner/local/commands.py
+++ b/tools/testrunner/local/commands.py
@@ -50,7 +50,7 @@
return prev_error_mode
-def RunProcess(verbose, timeout, args, additional_env, **rest):
+def RunProcess(verbose, timeout, args, **rest):
if verbose: print "#", " ".join(args)
popen_args = args
prev_error_mode = SEM_INVALID_VALUE
@@ -64,7 +64,6 @@
Win32SetErrorMode(error_mode | prev_error_mode)
env = os.environ.copy()
- env.update(additional_env)
# GTest shard information is read by the V8 tests runner. Make sure it
# doesn't leak into the execution of gtests we're wrapping. Those might
# otherwise apply a second level of sharding and as a result skip tests.
@@ -127,6 +126,6 @@
)
-def Execute(args, verbose=False, timeout=None, env=None):
+def Execute(args, verbose=False, timeout=None):
args = [ c for c in args if c != "" ]
- return RunProcess(verbose, timeout, args, env or {})
+ return RunProcess(verbose, timeout, args=args)
« no previous file with comments | « test/mjsunit/testcfg.py ('k') | tools/testrunner/local/execution.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698