Remove deprecated option --no-new-test-results from run-webkit-tests.

Remove deprecated option --no-new-test-results from run-webkit-tests.

Committed: https://crrev.com/c58fcdb5a500f764ed3c6a38b074887897b92e61
Cr-Commit-Position: refs/heads/master@{#428233}

[qyearsley, 2016-10-27 21:46:16]

qyearsley@chromium.org changed reviewers:
+ dpranke@chromium.org, ojan@chromium.org

[Dirk Pranke, 2016-10-27 22:13:11]

lgtm.

https://codereview.chromium.org/2454943003/diff/1/third_party/WebKit/Tools/Sc...
File third_party/WebKit/Tools/Scripts/webkit-tools-completion.sh (right):

https://codereview.chromium.org/2454943003/diff/1/third_party/WebKit/Tools/Sc...
third_party/WebKit/Tools/Scripts/webkit-tools-completion.sh:78: complete -o
default -W "--add-platform-exceptions --complex-text --configuration
--guard-malloc --help --http --ignore-tests --launch-safari --leaks
--merge-leak-depth --new-test-results --no-http --no-show-results
--no-sample-on-timeout --no-strip-editing-callbacks --pixel-tests --platform
--port --quiet --random --reset-results --results-directory --reverse --root
--sample-on-timeout --singly --skipped --slowest --strict
--strip-editing-callbacks --threaded --timeout --tolerance
--use-remote-links-to-tests --valgrind --verbose -1 -c -g -h -i -l -m -o -p -q
-t -v" run-webkit-tests
I suspect this file is completely unused. We should probably get rid of it.

[qyearsley, 2016-10-28 00:11:55]

On 2016/10/27 at 22:13:11, dpranke wrote:
> lgtm.
> 
>
https://codereview.chromium.org/2454943003/diff/1/third_party/WebKit/Tools/Sc...
> File third_party/WebKit/Tools/Scripts/webkit-tools-completion.sh (right):
> 
>
https://codereview.chromium.org/2454943003/diff/1/third_party/WebKit/Tools/Sc...
> third_party/WebKit/Tools/Scripts/webkit-tools-completion.sh:78: complete -o
default -W "--add-platform-exceptions --complex-text --configuration
--guard-malloc --help --http --ignore-tests --launch-safari --leaks
--merge-leak-depth --new-test-results --no-http --no-show-results
--no-sample-on-timeout --no-strip-editing-callbacks --pixel-tests --platform
--port --quiet --random --reset-results --results-directory --reverse --root
--sample-on-timeout --singly --skipped --slowest --strict
--strip-editing-callbacks --threaded --timeout --tolerance
--use-remote-links-to-tests --valgrind --verbose -1 -c -g -h -i -l -m -o -p -q
-t -v" run-webkit-tests
> I suspect this file is completely unused. We should probably get rid of it.

Sounds reasonable. I notice that some options are auto-completed for me for some
commands but not others, but I'm not sure why; maybe webkit-tools-completion.sh
is useful, but I'm not sure.

[qyearsley, 2016-10-28 00:11:56]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-10-28 00:12:22]

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, 2016-10-28 01:37:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-28 01:40:26]

Description was changed from

==========
Remove deprecated option --no-new-test-results from run-webkit-tests.
==========

to

==========
Remove deprecated option --no-new-test-results from run-webkit-tests.

Committed: https://crrev.com/c58fcdb5a500f764ed3c6a38b074887897b92e61
Cr-Commit-Position: refs/heads/master@{#428233}
==========

[commit-bot: I haz the power, 2016-10-28 01:40:27]

Patchset 1 (id:??) landed as
https://crrev.com/c58fcdb5a500f764ed3c6a38b074887897b92e61
Cr-Commit-Position: refs/heads/master@{#428233}