media: Add MemoryDataSource.

media: Add MemoryDataSource.

This is mainly for testing purpose. It will be used by PipelineIntegrationTestBase to test data in memory. Currently PipelineIntegrationTestBase only supports test data in file.

BUG=584119
TEST=Added unittest.

Committed: https://crrev.com/95acbbaea5077dcd05ba8fce0451b71e7cbe8468
Cr-Commit-Position: refs/heads/master@{#378585}

[xhwang, 2016-03-01 17:21:21]

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

[xhwang, 2016-03-01 17:21:21]

PTAL

[xhwang, 2016-03-01 17:22:37]

Description was changed from

==========
media: Add MemoryDataSource.

This is mainly for testing purpose.

BUG=584119
TEST=Added unittest.
==========

to

==========
media: Add MemoryDataSource.

This is mainly for testing purpose. It will be used by
PipelineIntegrationTestBase to test data in memory. Currently
PipelineIntegrationTestBase only supports test data in file.

BUG=584119
TEST=Added unittest.
==========

[DaleCurtis, 2016-03-01 21:23:32]

lgtm, but I don't quite see the use to this. Are you going to store byte data in
.cc or .h files? Usually we would just read this from a file, hence
FileDataSource :)

[xhwang, 2016-03-01 21:33:47]

On 2016/03/01 21:23:32, DaleCurtis wrote:
> lgtm, but I don't quite see the use to this. Are you going to store byte data
in
> .cc or .h files? Usually we would just read this from a file, hence
> FileDataSource :)

This is in preparation of fuzzer test for pipeline, and I thought you proposed
to "change the PipelineIntegrationTestBase to replace the FileDataSource created
here with some sort of special in memory data source from the fuzzer" :)

https://bugs.chromium.org/p/chromium/issues/detail?id=584119

To see how this is used in PipelineIntegrationTestBase:
https://chromiumcodereview.appspot.com/1757593002/

To see how a media pipeline fuzzer test works:
https://chromiumcodereview.appspot.com/1757603002/

After all of these I'll upload test data for ffmpeg regression test so they can
be fed to the media pipeline test as initial input.

Please let me know if you have any concerns :)

[DaleCurtis, 2016-03-01 22:06:28]

Ahhh, sorry I'd forgotten about the fuzzer stuff. lgtm

[xhwang, 2016-03-01 22:39:40]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2016-03-01 22:40:51]

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

[commit-bot: I haz the power, 2016-03-01 22:50:32]

Description was changed from

==========
media: Add MemoryDataSource.

This is mainly for testing purpose. It will be used by
PipelineIntegrationTestBase to test data in memory. Currently
PipelineIntegrationTestBase only supports test data in file.

BUG=584119
TEST=Added unittest.
==========

to

==========
media: Add MemoryDataSource.

This is mainly for testing purpose. It will be used by
PipelineIntegrationTestBase to test data in memory. Currently
PipelineIntegrationTestBase only supports test data in file.

BUG=584119
TEST=Added unittest.
==========

[commit-bot: I haz the power, 2016-03-01 22:50:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-01 22:52:03]

Description was changed from

==========
media: Add MemoryDataSource.

This is mainly for testing purpose. It will be used by
PipelineIntegrationTestBase to test data in memory. Currently
PipelineIntegrationTestBase only supports test data in file.

BUG=584119
TEST=Added unittest.
==========

to

==========
media: Add MemoryDataSource.

This is mainly for testing purpose. It will be used by
PipelineIntegrationTestBase to test data in memory. Currently
PipelineIntegrationTestBase only supports test data in file.

BUG=584119
TEST=Added unittest.

Committed: https://crrev.com/95acbbaea5077dcd05ba8fce0451b71e7cbe8468
Cr-Commit-Position: refs/heads/master@{#378585}
==========

[commit-bot: I haz the power, 2016-03-01 22:52:06]

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