[serial] Cancel read watcher if immediate read succeeded.

Cancel read watcher if an immediate read succeeds. Otherwise the
pending_read_buffer can be overwritten by the read watcher when it triggers
before a queued up ReadCompleted() call.

BUG=634278
Committed: https://crrev.com/42b13d59919ca019e92b4decd0e7db5a636b1c32
Cr-Commit-Position: refs/heads/master@{#410148}

[aschulman, 2016-08-05 05:02:55]

Description was changed from

==========
[serial] Cancel read watcher if immediate read succeeded.

Cancel read watcher if an immediate read succeeds. Otherwise the
pending_read_buffer can be overwritten by the read watcher when it triggers
before a queued up ReadCompleted() call.

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

to

==========
[serial] Cancel read watcher if immediate read succeeded.

Cancel read watcher if an immediate read succeeds. Otherwise the
pending_read_buffer can be overwritten by the read watcher when it triggers
before a queued up ReadCompleted() call.

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

[aschulman, 2016-08-05 05:02:55]

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

[charliea (OOO until 10-5), 2016-08-05 16:07:08]

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

[charliea (OOO until 10-5), 2016-08-05 16:07:10]

nit: the CL subject shouldn't repeated in the first paragraph of the CL
description

non-owner lgtm

[Reilly Grant (use Gerrit), 2016-08-05 18:20:28]

lgtm

[aschulman, 2016-08-05 19:09:20]

Description was changed from

==========
[serial] Cancel read watcher if immediate read succeeded.

Cancel read watcher if an immediate read succeeds. Otherwise the
pending_read_buffer can be overwritten by the read watcher when it triggers
before a queued up ReadCompleted() call.

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

to

==========
Cancel read watcher if an immediate read succeeds. Otherwise the
pending_read_buffer can be overwritten by the read watcher when it triggers
before a queued up ReadCompleted() call.

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

[aschulman, 2016-08-05 19:09:39]

The CQ bit was checked by aschulman@chromium.org

[commit-bot: I haz the power, 2016-08-05 19:10:21]

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-05 19:48:01]

Description was changed from

==========
Cancel read watcher if an immediate read succeeds. Otherwise the
pending_read_buffer can be overwritten by the read watcher when it triggers
before a queued up ReadCompleted() call.

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

to

==========
Cancel read watcher if an immediate read succeeds. Otherwise the
pending_read_buffer can be overwritten by the read watcher when it triggers
before a queued up ReadCompleted() call.

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

[commit-bot: I haz the power, 2016-08-05 19:48:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-05 19:49:38]

Description was changed from

==========
Cancel read watcher if an immediate read succeeds. Otherwise the
pending_read_buffer can be overwritten by the read watcher when it triggers
before a queued up ReadCompleted() call.

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

to

==========
Cancel read watcher if an immediate read succeeds. Otherwise the
pending_read_buffer can be overwritten by the read watcher when it triggers
before a queued up ReadCompleted() call.

BUG=634278

Committed: https://crrev.com/42b13d59919ca019e92b4decd0e7db5a636b1c32
Cr-Commit-Position: refs/heads/master@{#410148}
==========

[commit-bot: I haz the power, 2016-08-05 19:49:39]

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

[aschulman, 2016-08-06 04:10:29]

On 2016/08/05 16:07:10, charliea wrote:
> nit: the CL subject shouldn't repeated in the first paragraph of the CL
> description
> 
> non-owner lgtm

whoops, I misinterpreted this comment and blew away the commit subject from the
description. I didn't realize that the first sentence of my description
contained the same information as the commit subject.