Plumb DemuxerStream::EnableBitstreamConverter through mojo pipeline.

Plumb DemuxerStream::EnableBitstreamConverter through mojo pipeline.

BUG=592114
Committed: https://crrev.com/198d663363a342e390d57e789558e66fd63ef800
Cr-Commit-Position: refs/heads/master@{#379856}

[alokp, 2016-03-08 00:19:56]

alokp@chromium.org changed reviewers:
+ xhwang@chromium.org

[xhwang, 2016-03-08 01:36:46]

LGTM % tiny nit

https://chromiumcodereview.appspot.com/1774763002/diff/1/media/mojo/interface...
File media/mojo/interfaces/demuxer_stream.mojom (right):

https://chromiumcodereview.appspot.com/1774763002/diff/1/media/mojo/interface...
media/mojo/interfaces/demuxer_stream.mojom:57: EnableBitstreamConverter();
Could you please add a comment.

[alokp, 2016-03-08 05:22:42]

https://chromiumcodereview.appspot.com/1774763002/diff/1/media/mojo/interface...
File media/mojo/interfaces/demuxer_stream.mojom (right):

https://chromiumcodereview.appspot.com/1774763002/diff/1/media/mojo/interface...
media/mojo/interfaces/demuxer_stream.mojom:57: EnableBitstreamConverter();
On 2016/03/08 01:36:46, xhwang wrote:
> Could you please add a comment.

I wanted to copy the comment from media::DemuxerStream, but did not find any
comment there. Can you suggest an appropriate comment? I am not too sure what
this function is supposed to do in generic terms.

[xhwang, 2016-03-08 06:34:27]

https://chromiumcodereview.appspot.com/1774763002/diff/1/media/mojo/interface...
File media/mojo/interfaces/demuxer_stream.mojom (right):

https://chromiumcodereview.appspot.com/1774763002/diff/1/media/mojo/interface...
media/mojo/interfaces/demuxer_stream.mojom:57: EnableBitstreamConverter();
On 2016/03/08 05:22:42, alokp wrote:
> On 2016/03/08 01:36:46, xhwang wrote:
> > Could you please add a comment.
> 
> I wanted to copy the comment from media::DemuxerStream, but did not find any
> comment there. Can you suggest an appropriate comment? I am not too sure what
> this function is supposed to do in generic terms.

Fair enough :) How about:

Enables converting bitstream to a format that is expected by the decoder. For
example, H.264 bitstream based packets into H.264 Annex B bytestream format.

[alokp, 2016-03-08 17:25:53]

The CQ bit was checked by alokp@chromium.org

[alokp, 2016-03-08 17:25:53]

The patchset sent to the CQ was uploaded after l-g-t-m from xhwang@chromium.org
Link to the patchset: https://codereview.chromium.org/1774763002/#ps20001
(title: "added comment")

[commit-bot: I haz the power, 2016-03-08 17:26:06]

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

[commit-bot: I haz the power, 2016-03-08 18:06:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-08 18:07:24]

Description was changed from

==========
Plumb DemuxerStream::EnableBitstreamConverter through mojo pipeline.

BUG=592114
==========

to

==========
Plumb DemuxerStream::EnableBitstreamConverter through mojo pipeline.

BUG=592114

Committed: https://crrev.com/198d663363a342e390d57e789558e66fd63ef800
Cr-Commit-Position: refs/heads/master@{#379856}
==========

[commit-bot: I haz the power, 2016-03-08 18:07:25]

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