Title: [114441] trunk/Tools
Revision
114441
Author
dpra...@chromium.org
Date
2012-04-17 15:04:30 -0700 (Tue, 17 Apr 2012)

Log Message

rename --experimental-fully-parallel
https://bugs.webkit.org/show_bug.cgi?id=84188

Reviewed by Ojan Vafai.

I thought this was done long ago; rename
--experimental-fully-parallel to just --fully-parallel and
remove the --no- switch that AFAIK no one uses.

* Scripts/webkitpy/layout_tests/controllers/manager.py:
(Manager._run_tests):
* Scripts/webkitpy/layout_tests/run_webkit_tests.py:
(parse_args):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (114440 => 114441)


--- trunk/Tools/ChangeLog	2012-04-17 22:03:45 UTC (rev 114440)
+++ trunk/Tools/ChangeLog	2012-04-17 22:04:30 UTC (rev 114441)
@@ -1,3 +1,19 @@
+2012-04-17  Dirk Pranke  <dpra...@chromium.org>
+
+        rename --experimental-fully-parallel
+        https://bugs.webkit.org/show_bug.cgi?id=84188
+
+        Reviewed by Ojan Vafai.
+
+        I thought this was done long ago; rename
+        --experimental-fully-parallel to just --fully-parallel and
+        remove the --no- switch that AFAIK no one uses.
+
+        * Scripts/webkitpy/layout_tests/controllers/manager.py:
+        (Manager._run_tests):
+        * Scripts/webkitpy/layout_tests/run_webkit_tests.py:
+        (parse_args):
+
 2012-04-17  Christophe Dumez  <christophe.du...@intel.com>
 
         EFL's LayoutTestController does not implement callShouldCloseOnWebView

Modified: trunk/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py (114440 => 114441)


--- trunk/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py	2012-04-17 22:03:45 UTC (rev 114440)
+++ trunk/Tools/Scripts/webkitpy/layout_tests/controllers/manager.py	2012-04-17 22:04:30 UTC (rev 114441)
@@ -733,7 +733,7 @@
         thread_timings = []
 
         self._printer.print_update('Sharding tests ...')
-        locked_shards, unlocked_shards = self._shard_tests(file_list, int(self._options.child_processes), self._options.experimental_fully_parallel)
+        locked_shards, unlocked_shards = self._shard_tests(file_list, int(self._options.child_processes), self._options.fully_parallel)
 
         # FIXME: We don't have a good way to coordinate the workers so that
         # they don't try to run the shards that need a lock if we don't actually

Modified: trunk/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py (114440 => 114441)


--- trunk/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py	2012-04-17 22:03:45 UTC (rev 114440)
+++ trunk/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py	2012-04-17 22:04:30 UTC (rev 114441)
@@ -381,13 +381,8 @@
         optparse.make_option("--child-processes",
             help="Number of DumpRenderTrees to run in parallel."),
         # FIXME: Display default number of child processes that will run.
-        optparse.make_option("-f", "--experimental-fully-parallel",
-            action=""
+        optparse.make_option("-f", "--fully-parallel", action=""
             help="run all tests in parallel"),
-        optparse.make_option("--no-experimental-fully-parallel",
-            action=""
-            dest="experimental_fully_parallel",
-            help="do not run all tests in parallel"),
         optparse.make_option("--exit-after-n-failures", type="int", default=500,
             help="Exit after the first N failures instead of running all "
             "tests"),
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to