[devil] Work around occasionally fatal `adb forward --list` bug.

[devil] Work around occasionally fatal `adb forward --list` bug.

Additional context: b/31811775

BUG=chromium:623989
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/984fd95d0458d37eff56be2f4195223ffc18dfa3

[jbudorick, 2016-10-24 02:37:23]

jbudorick@chromium.org changed reviewers:
+ nednguyen@google.com, stip@chromium.org

[nednguyen, 2016-10-24 12:05:50]

lgtm

https://codereview.chromium.org/2447583002/diff/20001/devil/devil/android/sdk...
File devil/devil/android/sdk/adb_wrapper.py (right):

https://codereview.chromium.org/2447583002/diff/20001/devil/devil/android/sdk...
devil/devil/android/sdk/adb_wrapper.py:651: # once b/31811775 is fixed.
Should we remove the bug reference since this is internal?

https://codereview.chromium.org/2447583002/diff/20001/telemetry/telemetry/int...
File telemetry/telemetry/internal/platform/android_platform_backend.py (right):

https://codereview.chromium.org/2447583002/diff/20001/telemetry/telemetry/int...
telemetry/telemetry/internal/platform/android_platform_backend.py:499: #
starting in adb 1.0.36, a bug (b/31811775) keeps this from working.
same here

[jbudorick, 2016-10-24 13:46:16]

https://codereview.chromium.org/2447583002/diff/20001/devil/devil/android/sdk...
File devil/devil/android/sdk/adb_wrapper.py (right):

https://codereview.chromium.org/2447583002/diff/20001/devil/devil/android/sdk...
devil/devil/android/sdk/adb_wrapper.py:651: # once b/31811775 is fixed.
On 2016/10/24 12:05:50, nednguyen wrote:
> Should we remove the bug reference since this is internal?

I'd definitely like to leave it here. It's certainly not our first b/ link:
https://codesearch.chromium.org/search/?q=%5C+b%5C/%5Cd%2B&sq=package:chromiu...

https://codereview.chromium.org/2447583002/diff/20001/telemetry/telemetry/int...
File telemetry/telemetry/internal/platform/android_platform_backend.py (right):

https://codereview.chromium.org/2447583002/diff/20001/telemetry/telemetry/int...
telemetry/telemetry/internal/platform/android_platform_backend.py:499: #
starting in adb 1.0.36, a bug (b/31811775) keeps this from working.
On 2016/10/24 12:05:50, nednguyen wrote:
> same here

I don't mind removing it from here if you'd prefer, though.

[nednguyen, 2016-10-24 13:54:34]

On 2016/10/24 13:46:16, jbudorick wrote:
>
https://codereview.chromium.org/2447583002/diff/20001/devil/devil/android/sdk...
> File devil/devil/android/sdk/adb_wrapper.py (right):
> 
>
https://codereview.chromium.org/2447583002/diff/20001/devil/devil/android/sdk...
> devil/devil/android/sdk/adb_wrapper.py:651: # once b/31811775 is fixed.
> On 2016/10/24 12:05:50, nednguyen wrote:
> > Should we remove the bug reference since this is internal?
> 
> I'd definitely like to leave it here. It's certainly not our first b/ link:
>
https://codesearch.chromium.org/search/?q=%5C+b%5C/%5Cd%2B&sq=package:chromiu...
> 
>
https://codereview.chromium.org/2447583002/diff/20001/telemetry/telemetry/int...
> File telemetry/telemetry/internal/platform/android_platform_backend.py
(right):
> 
>
https://codereview.chromium.org/2447583002/diff/20001/telemetry/telemetry/int...
> telemetry/telemetry/internal/platform/android_platform_backend.py:499: #
> starting in adb 1.0.36, a bug (b/31811775) keeps this from working.
> On 2016/10/24 12:05:50, nednguyen wrote:
> > same here
> 
> I don't mind removing it from here if you'd prefer, though.

It's fine to keep the bug links then, lgtm

[jbudorick, 2016-10-24 13:56:08]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2016-10-24 13:56:17]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-24 14:19:30]

Description was changed from

==========
[devil] Work around occasionally fatal `adb forward --list` bug.

Additional context: b/31811775

BUG=chromium:623989
==========

to

==========
[devil] Work around occasionally fatal `adb forward --list` bug.

Additional context: b/31811775

BUG=chromium:623989

Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2016-10-24 14:19:31]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...