Minor cleanup of some recipe framework code.

scripts/slave/recipe_modules/step/api.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from slave import recipe_api
 from slave import recipe_util
 
 class StepApi(recipe_api.RecipeApi):
-  def __init__(self, *args, **kwargs):
+  def __init__(self, **kwargs):
+    super(StepApi, self).__init__(**kwargs)

[Vadim Sh., 2014/03/05 06:09:30]

This seems to be more in line with other modules. 

[iannucci, 2014/03/05 20:43:37]

Yeah, no reason not to.

     self._auto_resolve_conflicts = False
     self._name_function = None
     self._step_names = {}
-    super(StepApi, self).__init__(*args, **kwargs)
 
   # Making these properties makes them show up in show_me_the_modules,
   # and also makes it clear that they are intended to be mutated.
   @property
   def auto_resolve_conflicts(self):
     """Automatically resolve step name conflicts."""
     return self._auto_resolve_conflicts
 
   @auto_resolve_conflicts.setter
   def auto_resolve_conflicts(self, val):
@@ skipping 26 matching lines @@
       step_count = self._step_names.setdefault(name, 0) + 1
       self._step_names[name] = step_count
       if step_count > 1:
         name = "%s (%d)" % (name, step_count)
     kwargs.update({'name': name, 'cmd': cmd})
 
     schema = self.make_config()
     schema.set_val(kwargs)
 
     return schema.as_jsonish()