Android: a few minor improvements for the test runner.

Android: a few minor improvements for the test runner.

- We had a few options with default=True, which made them not really
optional. Instead, use an option with action='store_true' which is False by
default.
- Uses buildbot_report.py rather than duplicate the functionality.
- Removes default params from SingleTestRunner to make it more explicit.

BUG=
TEST=existing android tests.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=146318

[John Knottenbelt, 2012-07-11 17:02:13]

lgtm, thanks Marcus! One inline comment where I think you meant PrintError()
instead of Error()

http://codereview.chromium.org/10689159/diff/1/build/android/run_tests.py
File build/android/run_tests.py (right):

http://codereview.chromium.org/10689159/diff/1/build/android/run_tests.py#new...
build/android/run_tests.py:173: buildbot_report.Error()
Shouldn't this be .PrintError()?

[bulach, 2012-07-11 17:12:22]

thanks john! comment addressed..

http://codereview.chromium.org/10689159/diff/1/build/android/run_tests.py
File build/android/run_tests.py (right):

http://codereview.chromium.org/10689159/diff/1/build/android/run_tests.py#new...
build/android/run_tests.py:173: buildbot_report.Error()
On 2012/07/11 17:02:13, John Knottenbelt wrote:
> Shouldn't this be .PrintError()?

good catch! done.

[commit-bot: I haz the power, 2012-07-12 07:31:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/10689159/8001

[commit-bot: I haz the power, 2012-07-12 09:00:43]

Try job failure for 10689159-8001 (retry) (retry) on mac_rel for step
"browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-07-12 09:22:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/10689159/8001

[commit-bot: I haz the power, 2012-07-12 10:19:51]

Try job failure for 10689159-8001 (retry) on mac_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-07-12 10:27:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/10689159/8001

[commit-bot: I haz the power, 2012-07-12 11:35:05]

Change committed as 146318