Add suite specific options argument to Android test bot.

build/android/buildbot/bb_device_steps.py

Index: build/android/buildbot/bb_device_steps.py
diff --git a/build/android/buildbot/bb_device_steps.py b/build/android/buildbot/bb_device_steps.py
index 820074977591e681ffb5e59379565b5b32cce946..73461b7e8c112d0f7953fcafee5ffa446f63a398 100755
--- a/build/android/buildbot/bb_device_steps.py
+++ b/build/android/buildbot/bb_device_steps.py
@@ -115,6 +115,9 @@ def RunTestSuites(options, suites):
     cmd = ['build/android/test_runner.py', 'gtest', '-s', suite] + args
     if suite == 'content_browsertests':
       cmd.append('--num_retries=1')
+    elif suite == 'clank_unittests':
+      cmd += ['--isolate_file_path',
+              '"./clank/native/framework/clank_unittests.isolate"']

[jbudorick, 2014/05/21 23:31:04]

I don't think we want this hard-coded upstream (or downstream, for that matter).

This should either be accomplished by an extra argument to RunTestSuites, as
you've mentioned, or by adding to options, e.g.:

if options.isolate_file:
  args.extend(['--isolate_file_path', options.isolate_file])

[Kibeom Kim (inactive), 2014/05/21 23:41:24]

options argument seems like a global option that is being applied to all the
suites passed in, so instead I added suites_options.

Here is how downstream calls it
https://chrome-internal-review.googlesource.com/#/c/164153/2/build/buildbot/b...
.

Actually, this doesn't look that ideal since clank_unittests always requires the
--isolate_file_path option and it's better to be tied together.

     RunCmd(cmd)