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

third_party/WebKit/Tools/Scripts/webkitpy/tool/multi_command_tool.py

 # Copyright (c) 2009 Google Inc. All rights reserved.
 # Copyright (c) 2009 Apple Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
@@ skipping 17 matching lines @@
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 #
 # MultiCommandTool provides a framework for writing svn-like/git-like tools
 # which are called with the following format:
 # tool-name [global options] command-name [command options]
 
 import optparse
 import logging
 import sys
 
-from webkitpy.tool.commands.command import Command
 from webkitpy.tool.commands.command import HelpPrintingOptionParser
 from webkitpy.tool.commands.help_command import HelpCommand
 
 _log = logging.getLogger(__name__)
 
 
 class MultiCommandTool(object):
 
     global_options = None
 
-    def __init__(self, name=None, commands=None):
-        self._name = name or optparse.OptionParser(prog=name).get_prog_name()  # OptionParser has nice logic for fetching the name.
-        # Allow the unit tests to disable command auto-discovery.
-        self.commands = commands or [cls() for cls in self._find_all_commands() if cls.name]
+    def __init__(self, commands):
+        self.commands = commands
         self.help_command = self.command_by_name(HelpCommand.name)
-        # Require a help command, even if the manual test list doesn't include one.
+        # Require the help command, even if the manual test list doesn't include one.
         if not self.help_command:
             self.help_command = HelpCommand()
             self.commands.append(self.help_command)
+        # FIXME: Since tool is passed to Command.execute, it may not be necessary to set a tool attribute on the
+        # command objects here - maybe this should be done inside of Command.execute for commands that use self._tool.
         for command in self.commands:
             command.bind_to_tool(self)
 
-    @classmethod
-    def _add_all_subclasses(cls, class_to_crawl, seen_classes):
-        for subclass in class_to_crawl.__subclasses__():
-            if subclass not in seen_classes:
-                seen_classes.add(subclass)
-                cls._add_all_subclasses(subclass, seen_classes)
-
-    @classmethod
-    def _find_all_commands(cls):
-        commands = set()
-        cls._add_all_subclasses(Command, commands)
-        return sorted(commands)
-
     def name(self):
-        return self._name
+        return optparse.OptionParser().get_prog_name()
 
     def _create_option_parser(self):
         usage = "Usage: %prog [options] COMMAND [ARGS]"
         return HelpPrintingOptionParser(epilog_method=self.help_command._help_epilog, prog=self.name(), usage=usage)
 
     @staticmethod
     def _split_command_name_from_args(args):
         # Assume the first argument which doesn't start with "-" is the command name.
         command_index = 0
         for arg in args:
@@ skipping 49 matching lines @@
 
         (should_execute, failure_reason) = self.should_execute_command(command)
         if not should_execute:
             _log.error(failure_reason)
             return 0  # FIXME: Should this really be 0?
 
         result = command.check_arguments_and_execute(options, args, self)
 
         self.command_completed()
         return result