[Android]: Only kill 'adb' during device status check, not all commands with adb in command line.

[Android]: Only kill 'adb' during device status check, not all commands with adb in command line.

This was causing the perf tests to fail since the bb_device_steps_internal.py
had '--adb-root' as an argument and so was being accidentially killed during
the device status check.

BUG=312285
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=232610

[rmcilroy, 2013-11-01 15:52:56]

navabi: please take a look - this bug is currently preventing any of the
downstream Android perf bots from running.

[Isaac (away), 2013-11-01 19:13:38]

I added a comment on the original CL.  https://codereview.chromium.org/39833003/

This whole section seems problematic.

[navabi, 2013-11-01 20:25:40]

On 2013/11/01 19:13:38, Isaac wrote:
> I added a comment on the original CL. 
https://codereview.chromium.org/39833003/
> 
> This whole section seems problematic.

Example failure:
https://chromegw.corp.google.com/i/clank/builders/mako-sharded-official-perf-...

Based on ilevy's comments, it may make more sense to revert the original CL

Have you tested this locally and found that it fixes the problem?

[rmcilroy, 2013-11-01 21:58:12]

On 2013/11/01 20:25:40, navabi wrote:
> On 2013/11/01 19:13:38, Isaac wrote:
> > I added a comment on the original CL. 
> https://codereview.chromium.org/39833003/
> > 
> > This whole section seems problematic.
> 
> Example failure:
>
https://chromegw.corp.google.com/i/clank/builders/mako-sharded-official-perf-...
> 
> Based on ilevy's comments, it may make more sense to revert the original CL
> 
> Have you tested this locally and found that it fixes the problem?

Yes, I tested this locally and it fixed the problem.

[Isaac (away), 2013-11-01 22:28:41]

This doesn't make the code worse, so LGTM.  It's possible it will reduce the
utility of the original CL, but one step at a time.

[navabi, 2013-11-01 22:37:19]

On 2013/11/01 22:28:41, Isaac wrote:
> This doesn't make the code worse, so LGTM.  It's possible it will reduce the
> utility of the original CL, but one step at a time.

lgtm

[commit-bot: I haz the power, 2013-11-01 22:57:48]

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

[commit-bot: I haz the power, 2013-11-02 02:26:29]

List of reviewers changed. bulach@chromium.org,tonyg@chromium.org did a drive-by
without LGTM'ing!

[commit-bot: I haz the power, 2013-11-02 06:10:29]

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

[commit-bot: I haz the power, 2013-11-02 13:49:00]

Change committed as 232610