Merge webkit_patch and multi_command_tool.

Merge webkit_patch and multi_command_tool.

Since there is only one sub-class for MultiCommandTool, and we don't
plan on adding any more, removing MultiCommandTool simplifies the code
and makes it clearer that there's no more multi-command tools besides
webkit-patch.

This CL also updates the test accordingly -- some of the test methods
in multi_command_tool_unittest.py are not relevant if there's no general
MultiCommandTool superclass, so some test methods were removed.

BUG=621181
Committed: https://crrev.com/cb0ae443eb2805ad5ec09ff10f6488798d8ca421
Cr-Commit-Position: refs/heads/master@{#403615}

[qyearsley, 2016-06-30 00:04:47]

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

[Dirk Pranke, 2016-07-01 22:55:42]

lgtm w/ a couple of nits.

https://codereview.chromium.org/2110893003/diff/20001/third_party/WebKit/Tool...
File third_party/WebKit/Tools/Scripts/webkit-patch (left):

https://codereview.chromium.org/2110893003/diff/20001/third_party/WebKit/Tool...
third_party/WebKit/Tools/Scripts/webkit-patch:41: import
webkitpy.common.version_check
This line needs to stay in. Look at the module to see what it's doing.

https://codereview.chromium.org/2110893003/diff/20001/third_party/WebKit/Tool...
File third_party/WebKit/Tools/Scripts/webkitpy/tool/webkit_patch.py (right):

https://codereview.chromium.org/2110893003/diff/20001/third_party/WebKit/Tool...
third_party/WebKit/Tools/Scripts/webkitpy/tool/webkit_patch.py:81:
Host.__init__(self)
I'd probably use super() here to be consistent w/ most subclasses.

[qyearsley, 2016-07-02 17:26:30]

https://codereview.chromium.org/2110893003/diff/20001/third_party/WebKit/Tool...
File third_party/WebKit/Tools/Scripts/webkit-patch (left):

https://codereview.chromium.org/2110893003/diff/20001/third_party/WebKit/Tool...
third_party/WebKit/Tools/Scripts/webkit-patch:41: import
webkitpy.common.version_check
On 2016/07/01 at 22:55:42, Dirk Pranke wrote:
> This line needs to stay in. Look at the module to see what it's doing.

Done

https://codereview.chromium.org/2110893003/diff/20001/third_party/WebKit/Tool...
File third_party/WebKit/Tools/Scripts/webkitpy/tool/webkit_patch.py (right):

https://codereview.chromium.org/2110893003/diff/20001/third_party/WebKit/Tool...
third_party/WebKit/Tools/Scripts/webkitpy/tool/webkit_patch.py:81:
Host.__init__(self)
On 2016/07/01 at 22:55:42, Dirk Pranke wrote:
> I'd probably use super() here to be consistent w/ most subclasses.

Done

[qyearsley, 2016-07-02 17:26:42]

The CQ bit was checked by qyearsley@chromium.org

[qyearsley, 2016-07-02 17:26:43]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/2110893003/#ps60001
(title: "Sort imports; remove out-of-date comment.")

[commit-bot: I haz the power, 2016-07-02 17:26:54]

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-07-02 18:43:18]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-02 18:43:21]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-02 18:45:08]

Description was changed from

==========
Merge webkit_patch and multi_command_tool.

Since there is only one sub-class for MultiCommandTool, and we don't
plan on adding any more, removing MultiCommandTool simplifies the code
and makes it clearer that there's no more multi-command tools besides
webkit-patch.

This CL also updates the test accordingly -- some of the test methods
in multi_command_tool_unittest.py are not relevant if there's no general
MultiCommandTool superclass, so some test methods were removed.

BUG=621181
==========

to

==========
Merge webkit_patch and multi_command_tool.

Since there is only one sub-class for MultiCommandTool, and we don't
plan on adding any more, removing MultiCommandTool simplifies the code
and makes it clearer that there's no more multi-command tools besides
webkit-patch.

This CL also updates the test accordingly -- some of the test methods
in multi_command_tool_unittest.py are not relevant if there's no general
MultiCommandTool superclass, so some test methods were removed.

BUG=621181

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

[commit-bot: I haz the power, 2016-07-02 18:45:10]

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