Make run_webkit_tests random order by default for all platforms

Make run_webkit_tests random order by default for all platforms.

BUG=671805
R=dpranke@chromium.org,qyearsley@chromium.org

Committed: https://crrev.com/86c477efe2039ae9f9970313769f337d8606ca95
Cr-Commit-Position: refs/heads/master@{#441424}

[jeffcarp, 2016-12-20 23:56:27]

Here's the CL to move everything to random by default! I plan on merging this
once Windows is stable.

[qyearsley, 2016-12-20 23:58:06]

Excellent, LGTM :-)

https://codereview.chromium.org/2591933002/diff/1/third_party/WebKit/Tools/Sc...
File third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
(right):

https://codereview.chromium.org/2591933002/diff/1/third_party/WebKit/Tools/Sc...
third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py:332:
default=("random"),
Nit: No need for parentheses here.

[jeffcarp, 2016-12-20 23:59:14]

Description was changed from

==========
Make run_webkit_tests random order by default for all platforms

BUG=671805
R=dpranke@chromium.org,qyearsley@chromium.org
==========

to

==========
Make run_webkit_tests random order by default for all platforms.

BUG=671805
R=dpranke@chromium.org,qyearsley@chromium.org
==========

[jeffcarp, 2017-01-04 00:08:17]

The CQ bit was checked by jeffcarp@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 00:08:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-04 01:38:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-04 01:38:58]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jeffcarp, 2017-01-04 02:10:27]

On 2016/12/20 at 23:58:06, qyearsley wrote:
> Excellent, LGTM :-)
> 
>
https://codereview.chromium.org/2591933002/diff/1/third_party/WebKit/Tools/Sc...
> File
third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
(right):
> 
>
https://codereview.chromium.org/2591933002/diff/1/third_party/WebKit/Tools/Sc...
>
third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py:332:
default=("random"),
> Nit: No need for parentheses here.

Removed unnecessary parenthesis and also fixed some unit tests that had assumed
natural order by default.

[jeffcarp, 2017-01-04 02:10:30]

The CQ bit was checked by jeffcarp@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 02:10:46]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-04 04:07:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-04 04:07:53]

Dry run: This issue passed the CQ dry run.

[jeffcarp, 2017-01-04 19:04:29]

The CQ bit was checked by jeffcarp@chromium.org

[jeffcarp, 2017-01-04 19:04:30]

The patchset sent to the CQ was uploaded after l-g-t-m from
qyearsley@chromium.org
Link to the patchset: https://codereview.chromium.org/2591933002/#ps20001
(title: "Remove unnecessary parenthesis")

[commit-bot: I haz the power, 2017-01-04 19:05:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-04 19:10:12]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483556669972050,
"parent_rev": "78b554c8b269eb6a2341c1cf7fd2c0c117dc4c78", "commit_rev":
"4bc51990d1cea583d9ccdd6e00ea6b572739a682"}

[commit-bot: I haz the power, 2017-01-04 19:10:46]

Description was changed from

==========
Make run_webkit_tests random order by default for all platforms.

BUG=671805
R=dpranke@chromium.org,qyearsley@chromium.org
==========

to

==========
Make run_webkit_tests random order by default for all platforms.

BUG=671805
R=dpranke@chromium.org,qyearsley@chromium.org

Review-Url: https://codereview.chromium.org/2591933002
==========

[commit-bot: I haz the power, 2017-01-04 19:10:47]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-04 19:12:26]

Description was changed from

==========
Make run_webkit_tests random order by default for all platforms.

BUG=671805
R=dpranke@chromium.org,qyearsley@chromium.org

Review-Url: https://codereview.chromium.org/2591933002
==========

to

==========
Make run_webkit_tests random order by default for all platforms.

BUG=671805
R=dpranke@chromium.org,qyearsley@chromium.org

Committed: https://crrev.com/86c477efe2039ae9f9970313769f337d8606ca95
Cr-Commit-Position: refs/heads/master@{#441424}
==========

[commit-bot: I haz the power, 2017-01-04 19:12:29]

Patchset 2 (id:??) landed as
https://crrev.com/86c477efe2039ae9f9970313769f337d8606ca95
Cr-Commit-Position: refs/heads/master@{#441424}

[qyearsley, 2017-01-04 19:22:13]

On 2017/01/04 at 19:12:29, commit-bot wrote:
> Patchset 2 (id:??) landed as
https://crrev.com/86c477efe2039ae9f9970313769f337d8606ca95
> Cr-Commit-Position: refs/heads/master@{#441424}

\o/

[jeffcarp, 2017-01-04 20:59:19]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2617563002/ by jeffcarp@chromium.org.

The reason for reverting is: fast/text/ellipsis-stroked.html started failing on
some Linux and Mac bots..