Add EOF signal in FilterSourceStream::FilterData()

Add EOF signal in FilterSourceStream::FilterData()

Add a bool argument in FilterData() to signal that upstream's eof has been
reached.

R=rdsmith@chromium.org

BUG=474859
Committed: https://crrev.com/9e0e8b2bd96ede88c633fe62fe24dc20c24559f7
Cr-Commit-Position: refs/heads/master@{#418713}

[xunjieli, 2016-09-13 20:14:16]

Patchset #1 (id:1) has been deleted

[xunjieli, 2016-09-13 20:14:47]

Description was changed from

==========
Add EOF signal in FilterSourceStream::FilterData()

Add a bool argument in FilterData() to signal that upstream's eof has been
reached.

R=rdsmith@chromium.org
CC=mmenke@chromium.org

BUG=474859
==========

to

==========
Add EOF signal in FilterSourceStream::FilterData()

Add a bool argument in FilterData() to signal that upstream's eof has been
reached.

R=rdsmith@chromium.org

BUG=474859
==========

[xunjieli, 2016-09-13 20:16:22]

Thanks for the feedback on the earlier CL. Sorry for misunderstanding some of
the suggestions. This is to add the eof signal as suggested in the review
comment. PTAL. 
(I will do a separate CL to change the DrainableIOBuffer argument to IOBuffer
and &size)

[xunjieli, 2016-09-13 20:16:29]

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

[commit-bot: I haz the power, 2016-09-13 20:16:48]

Dry run: 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-09-13 21:26:54]

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

[commit-bot: I haz the power, 2016-09-13 21:26:55]

Dry run: This issue passed the CQ dry run.

[Randy Smith (Not in Mondays), 2016-09-14 21:21:20]

LGTM (ignore my offline comment; after some thought, I think the comments before
the test were good enough).

[xunjieli, 2016-09-14 21:25:12]

On 2016/09/14 21:21:20, Randy Smith - Not in Fridays wrote:
> LGTM (ignore my offline comment; after some thought, I think the comments
before
> the test were good enough).

Thanks for the review! Much appreciated.

[xunjieli, 2016-09-14 21:25:17]

The CQ bit was checked by xunjieli@chromium.org

[commit-bot: I haz the power, 2016-09-14 21:25:52]

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-09-14 23:09:16]

Description was changed from

==========
Add EOF signal in FilterSourceStream::FilterData()

Add a bool argument in FilterData() to signal that upstream's eof has been
reached.

R=rdsmith@chromium.org

BUG=474859
==========

to

==========
Add EOF signal in FilterSourceStream::FilterData()

Add a bool argument in FilterData() to signal that upstream's eof has been
reached.

R=rdsmith@chromium.org

BUG=474859
==========

[commit-bot: I haz the power, 2016-09-14 23:09:17]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-09-14 23:13:08]

Description was changed from

==========
Add EOF signal in FilterSourceStream::FilterData()

Add a bool argument in FilterData() to signal that upstream's eof has been
reached.

R=rdsmith@chromium.org

BUG=474859
==========

to

==========
Add EOF signal in FilterSourceStream::FilterData()

Add a bool argument in FilterData() to signal that upstream's eof has been
reached.

R=rdsmith@chromium.org

BUG=474859

Committed: https://crrev.com/9e0e8b2bd96ede88c633fe62fe24dc20c24559f7
Cr-Commit-Position: refs/heads/master@{#418713}
==========

[commit-bot: I haz the power, 2016-09-14 23:13:09]

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