Update DataPipeConsumerDispatcher state when deserialized

Update DataPipeConsumerDispatcher state when deserialized

When transferring a data pipe consumer, it is possible that state update
messages have already arrived when deserializing it. This CL makes
DataPipeConsumerDispatcher handle such messages. Otherwise it may not notice
the messages possibly forever.

BUG=None

Committed: https://crrev.com/d429ca27a689cbcfdb91b026a3f70695e8733f42
Cr-Commit-Position: refs/heads/master@{#411707}

[yhirano, 2016-08-12 08:25:33]

The CQ bit was checked by yhirano@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-12 08:25:43]

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

[yhirano, 2016-08-12 09:02:45]

Description was changed from

==========
Update DataPipeConsumerDispatcher state when deserialized

When transferring a data pipe consumer, it is possible that state update
messages have already arrived when deserializing it. This CL makes
DataPipeConsumerDispatcher handle such messages. Otherwise it may not notice
the messages potentially forever.

BUG=None
==========

to

==========
Update DataPipeConsumerDispatcher state when deserialized

When transferring a data pipe consumer, it is possible that state update
messages have already arrived when deserializing it. This CL makes
DataPipeConsumerDispatcher handle such messages. Otherwise it may not notice
the messages possibly forever.

BUG=None
==========

[commit-bot: I haz the power, 2016-08-12 09:11:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-12 09:11:13]

Dry run: This issue passed the CQ dry run.

[yhirano, 2016-08-12 10:04:45]

yhirano@chromium.org changed reviewers:
+ jam@chromium.org, yzshen@chromium.org

[yhirano, 2016-08-12 10:04:46]

PTAL

Sorry this CL doesn't have a test. I don't know how to write mojo tests...

I would be very happy if I could land this CL shortly because this issue
is blocking https://codereview.chromium.org/1970693002/.

Thanks!

[yzshen1, 2016-08-12 14:00:00]

LGTM (but please wait for John's opinion)

[jam, 2016-08-12 17:50:15]

lgtm

[yhirano, 2016-08-12 17:51:35]

Thanks!

[yhirano, 2016-08-12 17:51:40]

The CQ bit was checked by yhirano@chromium.org

[commit-bot: I haz the power, 2016-08-12 17:52:02]

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-12 18:27:18]

Description was changed from

==========
Update DataPipeConsumerDispatcher state when deserialized

When transferring a data pipe consumer, it is possible that state update
messages have already arrived when deserializing it. This CL makes
DataPipeConsumerDispatcher handle such messages. Otherwise it may not notice
the messages possibly forever.

BUG=None
==========

to

==========
Update DataPipeConsumerDispatcher state when deserialized

When transferring a data pipe consumer, it is possible that state update
messages have already arrived when deserializing it. This CL makes
DataPipeConsumerDispatcher handle such messages. Otherwise it may not notice
the messages possibly forever.

BUG=None
==========

[commit-bot: I haz the power, 2016-08-12 18:27:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-12 18:28:43]

Description was changed from

==========
Update DataPipeConsumerDispatcher state when deserialized

When transferring a data pipe consumer, it is possible that state update
messages have already arrived when deserializing it. This CL makes
DataPipeConsumerDispatcher handle such messages. Otherwise it may not notice
the messages possibly forever.

BUG=None
==========

to

==========
Update DataPipeConsumerDispatcher state when deserialized

When transferring a data pipe consumer, it is possible that state update
messages have already arrived when deserializing it. This CL makes
DataPipeConsumerDispatcher handle such messages. Otherwise it may not notice
the messages possibly forever.

BUG=None

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

[commit-bot: I haz the power, 2016-08-12 18:28:44]

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

[Ken Rockot(use gerrit already), 2016-08-19 18:15:08]

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

[Ken Rockot(use gerrit already), 2016-08-19 18:15:09]

Just stumbled on this CL. Filed http://crbug.com/639363 for it.