[serial] Only add a watcher if if the immeidate read attempt fails.

[serial] Only add a watcher if if the immeidate read attempt fails.

BUG=634278
Committed: https://crrev.com/577a8466eb739efbe3ba7887e64f9b230f62de0c
Cr-Commit-Position: refs/heads/master@{#409917}

[aschulman, 2016-08-04 09:40:41]

Description was changed from

==========
[serial] Only add a watcher if if the immeidate read attempt fails.

BUG=634278
==========

to

==========
[serial] Only add a watcher if if the immeidate read attempt fails.

BUG=634278
==========

[aschulman, 2016-08-04 09:40:41]

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

[Reilly Grant (use Gerrit), 2016-08-04 17:55:33]

I think you could also move the call to EnsureWatchingReads() into the EAGAIN
case but this lgtm.

[aschulman, 2016-08-04 20:46:43]

The CQ bit was checked by aschulman@chromium.org

[aschulman, 2016-08-04 20:46:43]

The patchset sent to the CQ was uploaded after l-g-t-m from reillyg@chromium.org
Link to the patchset: https://codereview.chromium.org/2201893008/#ps20001
(title: "Moved to EAGAIN")

[commit-bot: I haz the power, 2016-08-04 20:47:16]

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-08-04 22:30:32]

Description was changed from

==========
[serial] Only add a watcher if if the immeidate read attempt fails.

BUG=634278
==========

to

==========
[serial] Only add a watcher if if the immeidate read attempt fails.

BUG=634278
==========

[commit-bot: I haz the power, 2016-08-04 22:30:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-04 22:33:49]

Description was changed from

==========
[serial] Only add a watcher if if the immeidate read attempt fails.

BUG=634278
==========

to

==========
[serial] Only add a watcher if if the immeidate read attempt fails.

BUG=634278

Committed: https://crrev.com/577a8466eb739efbe3ba7887e64f9b230f62de0c
Cr-Commit-Position: refs/heads/master@{#409917}
==========

[commit-bot: I haz the power, 2016-08-04 22:33:50]

Patchset 2 (id:??) landed as
https://crrev.com/577a8466eb739efbe3ba7887e64f9b230f62de0c
Cr-Commit-Position: refs/heads/master@{#409917}

[aschulman, 2016-08-05 04:05:16]

Description was changed from

==========
[serial] Only add a watcher if if the immeidate read attempt fails.

BUG=634278

Committed: https://crrev.com/577a8466eb739efbe3ba7887e64f9b230f62de0c
Cr-Commit-Position: refs/heads/master@{#409917}
==========

to

==========
[serial] Only add a watcher if if the immeidate read attempt fails.

BUG=634278

Committed: https://crrev.com/577a8466eb739efbe3ba7887e64f9b230f62de0c
Cr-Commit-Position: refs/heads/master@{#409917}
==========