isolate: add run_isolated, deprecate runtest

scripts/slave/recipe_modules/isolate/api.py

Index: scripts/slave/recipe_modules/isolate/api.py
diff --git a/scripts/slave/recipe_modules/isolate/api.py b/scripts/slave/recipe_modules/isolate/api.py
index e96b289d1928917bb5041e18532181789bb7cffd..0044778f9100517b1cd92f1dd8377d62f4064dd1 100644
--- a/scripts/slave/recipe_modules/isolate/api.py
+++ b/scripts/slave/recipe_modules/isolate/api.py
@@ -213,11 +213,25 @@ class IsolateApi(recipe_api.RecipeApi):
       full_args.extend(args)
     return full_args
 
+  def run_isolated(self, name, isolate_hash, args=None):
+    """Runs an isolated test."""
+    self.m.python(
+        name,
+        self._run_isolated_path,
+        [
+            '--isolated', isolate_hash,
+            '-I', self.isolate_server,
+            '--verbose',

[M-A Ruel, 2015/09/29 12:21:39]

Remove, unless it's to test out then remove later.

[Paweł Hajdan Jr., 2015/09/29 12:25:40]

It's intentional - and as discussed offline, my understanding is that's okay for
you.

+            '--',

[M-A Ruel, 2015/09/29 12:21:39]

Personally, I'd prefer '--' to not be added when args is empty;

cmd = [
...
]
if args:
  cmd.append('--')
  cmd.extend(args)

It'd be nicer IMHO but not a big deal.

[Paweł Hajdan Jr., 2015/09/29 12:25:40]

Done.

+        ] + (args or []))
+
   def runtest(self, test, revision, webkit_revision, args=None, name=None,
               **runtest_kwargs):
     """Runs a test which has previously been isolated to the server.
 
     Uses runtest_args_list, above, and delegates to api.chromium.runtest.
+
+    DEPRECATED - run_isolated above is strongly recommended for all new callers.
     """
     self.m.chromium.runtest(
         self._run_isolated_path,