Android: adds instrumentation tests on the FYI bot.

Android: adds instrumentation tests on the FYI bot.

BUG=139365
TEST=Android FYI tester runs instrumentation tests.


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

[bulach, 2012-07-30 21:37:59]

many thanks to jknotten and yusuf: everything else is already in place!
we just need to hook this up with the FYI bot..

[Yaron, 2012-07-30 21:41:53]

http://codereview.chromium.org/10826071/diff/1/build/android/buildbot_functio...
File build/android/buildbot_functions.sh (right):

http://codereview.chromium.org/10826071/diff/1/build/android/buildbot_functio...
build/android/buildbot_functions.sh:258: function bb_run_content_shell_test {
Can you remove this one? We've opted to replace it with real instrumentation
tests as this one is flaky and basically a temporary stand-in for what you're
doing.

[bulach, 2012-07-30 21:50:03]

http://codereview.chromium.org/10826071/diff/1/build/android/buildbot_functio...
File build/android/buildbot_functions.sh (right):

http://codereview.chromium.org/10826071/diff/1/build/android/buildbot_functio...
build/android/buildbot_functions.sh:258: function bb_run_content_shell_test {
On 2012/07/30 21:41:53, Yaron wrote:
> Can you remove this one? We've opted to replace it with real instrumentation
> tests as this one is flaky and basically a temporary stand-in for what you're
> doing.

this function is already called by the try bots, which I think runs for the CQ,
right?

would it be ok to do this cleanup as a separate step?

ideally, we'd leave this for a day or two, then graduate / replace the old one
once it proves stable.. how does it sound?

I mean, if it's already really flaky :) I'm happy to remove it right now, but
otherwise I'd prefer to be a bit more conservative..

[Yaron, 2012-07-30 21:56:00]

lgtm

follow-up is fine

[commit-bot: I haz the power, 2012-07-31 10:09:41]

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

[bulach, 2012-07-31 10:10:03]

thanks! CQing, and I'll send the follow up in a sec..

[commit-bot: I haz the power, 2012-07-31 11:47:18]

Change committed as 149171