Retry BattOr connection serial reads when zero bytes are read

Retry BattOr connection serial reads when zero bytes are read

The BattOr connection will now continue to make read attempts until the
timeout is imposed by the BattOrAgent by canceling the read.

This fixes a Windows bug in which the BattOr agent would request the
samples from the BattOr, immediately request a read, receive zero bytes
back, fail, and restart StopTracing.

TBR=aschulman@chromium.org
BUG=726087
Review-Url: https://codereview.chromium.org/2907793003
Cr-Commit-Position: refs/heads/master@{#475169}
Committed: https://chromium.googlesource.com/chromium/src/+/e08d9d7cc11ca18f127f900c0e76499fba723c99

[charliea (OOO until 10-5), 2017-05-26 21:27:56]

Description was changed from

==========
Retry BattOr connection serial reads when zero bytes were read

The BattOr connection will now continue to make read attempts until the
timeout is imposed by the BattOrAgent by canceling the read.

This fixes a Windows bug in which the BattOr agent would request the
samples from the BattOr, immediately request a read, receive zero bytes
back, fail, and restart StopTracing.

BUG=726087
==========

to

==========
Retry BattOr connection serial reads when zero bytes are read

The BattOr connection will now continue to make read attempts until the
timeout is imposed by the BattOrAgent by canceling the read.

This fixes a Windows bug in which the BattOr agent would request the
samples from the BattOr, immediately request a read, receive zero bytes
back, fail, and restart StopTracing.

BUG=726087
==========

[charliea (OOO until 10-5), 2017-05-26 21:28:07]

charliea@chromium.org changed reviewers:
+ aschulman@chromium.org

[charliea (OOO until 10-5), 2017-05-26 21:42:31]

Description was changed from

==========
Retry BattOr connection serial reads when zero bytes are read

The BattOr connection will now continue to make read attempts until the
timeout is imposed by the BattOrAgent by canceling the read.

This fixes a Windows bug in which the BattOr agent would request the
samples from the BattOr, immediately request a read, receive zero bytes
back, fail, and restart StopTracing.

BUG=726087
==========

to

==========
Retry BattOr connection serial reads when zero bytes are read

The BattOr connection will now continue to make read attempts until the
timeout is imposed by the BattOrAgent by canceling the read.

This fixes a Windows bug in which the BattOr agent would request the
samples from the BattOr, immediately request a read, receive zero bytes
back, fail, and restart StopTracing.

TBR=aschulman@chromium.org
BUG=726087
==========

[charliea (OOO until 10-5), 2017-05-26 21:42:44]

The CQ bit was checked by charliea@chromium.org

[commit-bot: I haz the power, 2017-05-26 21:43:25]

CQ is trying da patch.

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

[aschulman, 2017-05-26 21:43:45]

On 2017/05/26 21:43:25, commit-bot: I haz the power wrote:
> CQ is trying da patch.
> 
> Follow status at:
>
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

LGTM

[commit-bot: I haz the power, 2017-05-26 22:48:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495834964208830, "parent_rev":
"0f97c95ec5e806e2fe5dc2a6c33ec8d1c5aba2e4", "commit_rev":
"e08d9d7cc11ca18f127f900c0e76499fba723c99"}

[commit-bot: I haz the power, 2017-05-26 22:48:16]

Description was changed from

==========
Retry BattOr connection serial reads when zero bytes are read

The BattOr connection will now continue to make read attempts until the
timeout is imposed by the BattOrAgent by canceling the read.

This fixes a Windows bug in which the BattOr agent would request the
samples from the BattOr, immediately request a read, receive zero bytes
back, fail, and restart StopTracing.

TBR=aschulman@chromium.org
BUG=726087
==========

to

==========
Retry BattOr connection serial reads when zero bytes are read

The BattOr connection will now continue to make read attempts until the
timeout is imposed by the BattOrAgent by canceling the read.

This fixes a Windows bug in which the BattOr agent would request the
samples from the BattOr, immediately request a read, receive zero bytes
back, fail, and restart StopTracing.

TBR=aschulman@chromium.org
BUG=726087

Review-Url: https://codereview.chromium.org/2907793003
Cr-Commit-Position: refs/heads/master@{#475169}
Committed:
https://chromium.googlesource.com/chromium/src/+/e08d9d7cc11ca18f127f900c0e76...
==========

[commit-bot: I haz the power, 2017-05-26 22:48:17]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e08d9d7cc11ca18f127f900c0e76...