[webkit-reviews] review granted: [Bug 92805] nrwt: move sharding logic into layout_test_runner.py : [Attachment 155661] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jul 31 18:30:58 PDT 2012
Ryosuke Niwa <rniwa at webkit.org> has granted Dirk Pranke
<dpranke at chromium.org>'s request for review:
Bug 92805: nrwt: move sharding logic into layout_test_runner.py
https://bugs.webkit.org/show_bug.cgi?id=92805
Attachment 155661: Patch
https://bugs.webkit.org/attachment.cgi?id=155661&action=review
------- Additional Comments from Ryosuke Niwa <rniwa at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=155661&action=review
> Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py:370
> + 'locked_shard'),
Nit: wrong indentation.
> Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py:402
> + extract_and_flatten(some_shards)))
Nit: wrong indentation.
> Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py:407
> + def _dir_for_test_input(self, test_input):
> + """Returns the highest-level directory by which to shard the given
> + test file."""
I would rename the function to _highest_dir_to_shard_for_test_input and get rid
of the comment.
> Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py:411
> + # TODO(ojan): Make the http server on Windows be apache so we can
Nit: TODO -> FIXME.
> Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner.py:433
> +
http://www.codinghorror.com/blog/2007/12/sorting-for-humans-natural-sort-order.
html
> + http://nedbatchelder.com/blog/200712.html#e20071211T054956
Why are these URLs indented?
>
Tools/Scripts/webkitpy/layout_tests/controllers/layout_test_runner_unittest.py:
2
> +# Copyright (C) 2010 Google Inc. All rights reserved.
Maybe update the date here?
More information about the webkit-reviews
mailing list