Revert of Use an explicit list of webkit-patch commands instead of using auto-discovery.

Revert of Use an explicit list of webkit-patch commands instead of using auto-discovery. (patchset #2 id:20001 of https://codereview.chromium.org/2070333003/ )

Reason for revert:
Broke rebaseline-o-matic because commands are not bound to tool.

Original issue's description:
> Use an explicit list of webkit-patch commands instead of using auto-discovery.
>
> This is a followup to http://crrev.com/1950773002 and related CLs,
> which removed remove imports from other __init__.py files; after this,
> all of the __init__.py files in webkitpy are empty.
>
> Before this CL, webkit-patch relies on all commands being imported (by
> importing the package webkitpy.tool.commands) and then discovering all
> subclasses of command using the magic method __subclasses__.
>
> This CL makes it so that commands that are part of webkit-patch are
> explicitly listed, and there is no class discovery.
>
> BUG=598897
>
> Committed: https://crrev.com/b5277f6feda049f0941928ae198ed8fed41e15a2
> Cr-Commit-Position: refs/heads/master@{#400369}

TBR=dpranke@chromium.org,qyearsley@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=598897
Committed: https://crrev.com/e0afd0716fe7c94d890cf94b8771692bd827884e
Cr-Commit-Position: refs/heads/master@{#400376}

[Xianzhu, 2016-06-17 06:17:32]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-06-17 06:17:32]

Created Revert of Use an explicit list of webkit-patch commands instead of using
auto-discovery.

[commit-bot: I haz the power, 2016-06-17 06:17:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2075073002/1

[commit-bot: I haz the power, 2016-06-17 06:18:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-17 06:21:00]

Description was changed from

==========
Revert of Use an explicit list of webkit-patch commands instead of using
auto-discovery. (patchset #2 id:20001 of
https://codereview.chromium.org/2070333003/ )

Reason for revert:
Broke rebaseline-o-matic because commands are not bound to tool.

Original issue's description:
> Use an explicit list of webkit-patch commands instead of using auto-discovery.
> 
> This is a followup to http://crrev.com/1950773002 and related CLs,
> which removed remove imports from other __init__.py files; after this,
> all of the __init__.py files in webkitpy are empty.
> 
> Before this CL, webkit-patch relies on all commands being imported (by
> importing the package webkitpy.tool.commands) and then discovering all
> subclasses of command using the magic method __subclasses__.
> 
> This CL makes it so that commands that are part of webkit-patch are
> explicitly listed, and there is no class discovery.
> 
> BUG=598897
> 
> Committed: https://crrev.com/b5277f6feda049f0941928ae198ed8fed41e15a2
> Cr-Commit-Position: refs/heads/master@{#400369}

TBR=dpranke@chromium.org,qyearsley@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=598897
==========

to

==========
Revert of Use an explicit list of webkit-patch commands instead of using
auto-discovery. (patchset #2 id:20001 of
https://codereview.chromium.org/2070333003/ )

Reason for revert:
Broke rebaseline-o-matic because commands are not bound to tool.

Original issue's description:
> Use an explicit list of webkit-patch commands instead of using auto-discovery.
>
> This is a followup to http://crrev.com/1950773002 and related CLs,
> which removed remove imports from other __init__.py files; after this,
> all of the __init__.py files in webkitpy are empty.
>
> Before this CL, webkit-patch relies on all commands being imported (by
> importing the package webkitpy.tool.commands) and then discovering all
> subclasses of command using the magic method __subclasses__.
>
> This CL makes it so that commands that are part of webkit-patch are
> explicitly listed, and there is no class discovery.
>
> BUG=598897
>
> Committed: https://crrev.com/b5277f6feda049f0941928ae198ed8fed41e15a2
> Cr-Commit-Position: refs/heads/master@{#400369}

TBR=dpranke@chromium.org,qyearsley@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=598897

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

[commit-bot: I haz the power, 2016-06-17 06:21:01]

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

[Dirk Pranke, 2016-06-17 15:55:08]

lgtm

[qyearsley, 2016-06-17 16:19:22]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2073893004/ by qyearsley@chromium.org.

The reason for reverting is: I believe the problem was that the commands weren't
passed to the MultiCommandTool constructor, so
command.bind_to_tool(multi_command_tool) wasn't called for the commands..