[battor agent] Add 'read more bytes' logic in the BattOr connection

[battor agent] Add 'read more bytes' logic in the BattOr connection

Right now, when one read doesn't result in a complete message, the read
fails and retries up to RETRY_COUNT (3) times. However, the BattOr
connection SHOULD read more data from the serial connection until:

1) A complete message is read
2) Bytes are read that are irrecoverably erroneous in some way
or
3) More bytes are read than the maximum message length

This CL makes this the case.

BUG=613562
Committed: https://crrev.com/f7b11a03318b4e244bb743bb506afc702d7e8f7c
Cr-Commit-Position: refs/heads/master@{#395110}

[charliea (OOO until 10-5), 2016-05-20 15:19:10]

Description was changed from

==========
[battor agent] Adds 'read more bytes' logic in the BattOr connection

Right now, when one read doesn't result in a complete message, the read
fails and retries up to RETRY_COUNT (3) times. However, the BattOr
connection SHOULD read more data from the serial connection until:

1) A complete message is read
2) Bytes are read that are irrecoverably erroneous in some way
or
3) More bytes are read than the maximum message length

This CL makes this the case.

BUG=613562
==========

to

==========
[battor agent] Add 'read more bytes' logic in the BattOr connection

Right now, when one read doesn't result in a complete message, the read
fails and retries up to RETRY_COUNT (3) times. However, the BattOr
connection SHOULD read more data from the serial connection until:

1) A complete message is read
2) Bytes are read that are irrecoverably erroneous in some way
or
3) More bytes are read than the maximum message length

This CL makes this the case.

BUG=613562
==========

[charliea (OOO until 10-5), 2016-05-20 15:19:15]

Patchset #1 (id:1) has been deleted

[charliea (OOO until 10-5), 2016-05-20 15:19:30]

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

[aschulman, 2016-05-20 16:11:29]

lgtm

[aschulman, 2016-05-20 16:11:33]

lgtm

[charliea (OOO until 10-5), 2016-05-20 17:10:46]

The CQ bit was checked by charliea@chromium.org

[commit-bot: I haz the power, 2016-05-20 17:10:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2000743002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2000743002/20001

[commit-bot: I haz the power, 2016-05-20 17:53:09]

Description was changed from

==========
[battor agent] Add 'read more bytes' logic in the BattOr connection

Right now, when one read doesn't result in a complete message, the read
fails and retries up to RETRY_COUNT (3) times. However, the BattOr
connection SHOULD read more data from the serial connection until:

1) A complete message is read
2) Bytes are read that are irrecoverably erroneous in some way
or
3) More bytes are read than the maximum message length

This CL makes this the case.

BUG=613562
==========

to

==========
[battor agent] Add 'read more bytes' logic in the BattOr connection

Right now, when one read doesn't result in a complete message, the read
fails and retries up to RETRY_COUNT (3) times. However, the BattOr
connection SHOULD read more data from the serial connection until:

1) A complete message is read
2) Bytes are read that are irrecoverably erroneous in some way
or
3) More bytes are read than the maximum message length

This CL makes this the case.

BUG=613562
==========

[commit-bot: I haz the power, 2016-05-20 17:53:10]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-05-20 17:55:42]

Description was changed from

==========
[battor agent] Add 'read more bytes' logic in the BattOr connection

Right now, when one read doesn't result in a complete message, the read
fails and retries up to RETRY_COUNT (3) times. However, the BattOr
connection SHOULD read more data from the serial connection until:

1) A complete message is read
2) Bytes are read that are irrecoverably erroneous in some way
or
3) More bytes are read than the maximum message length

This CL makes this the case.

BUG=613562
==========

to

==========
[battor agent] Add 'read more bytes' logic in the BattOr connection

Right now, when one read doesn't result in a complete message, the read
fails and retries up to RETRY_COUNT (3) times. However, the BattOr
connection SHOULD read more data from the serial connection until:

1) A complete message is read
2) Bytes are read that are irrecoverably erroneous in some way
or
3) More bytes are read than the maximum message length

This CL makes this the case.

BUG=613562

Committed: https://crrev.com/f7b11a03318b4e244bb743bb506afc702d7e8f7c
Cr-Commit-Position: refs/heads/master@{#395110}
==========

[commit-bot: I haz the power, 2016-05-20 17:55:43]

Patchset 1 (id:??) landed as
https://crrev.com/f7b11a03318b4e244bb743bb506afc702d7e8f7c
Cr-Commit-Position: refs/heads/master@{#395110}