Reland Add MJPEG support to FakeVideoCaptureDevice

Reland Add MJPEG support to FakeVideoCaptureDevice

This was reverted as suspected to cause some ChromeOS Autotest failures, see 
crbug.com/696130. Those failures turned out to be very likely caused by 
something else.

The code is the same as in the original CL % rebase to ToT.

Original CL description:

Add MJPEG support to FakeVideoCaptureDevice

Part of a series of CLs for FakeVideoCaptureDevice with the goal of allowing us
to expand our test coverage of the video capture pipeline.

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL12.

BUG=584797
TEST=
  capture_unittests --gtest_filter="*Video*"

[1] https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxsxWslJ_X-zpOhijvI8/edit?usp=sharing

Review-Url: https://codereview.chromium.org/2721633006
Cr-Commit-Position: refs/heads/master@{#453726}
Committed: https://chromium.googlesource.com/chromium/src/+/fe4c57f8d526e1c286d0504f40e5b178cc527e6c

[chfremer, 2017-02-28 19:33:31]

Description was changed from

==========
Reland Add MJPEG support to FakeVideoCaptureDevice

BUG=
==========

to

==========
Reland Add MJPEG support to FakeVideoCaptureDevice

This was reverted as suspected to cause some ChromeOS Autotest failures, see 
crbug.com/696130. Those failures turned out to be very likely caused by 
something else.

The code is the same as in the original CL except for a one-line merge-in from 
CL https://codereview.chromium.org/2673373003/.

Original CL description:

Add MJPEG support to FakeVideoCaptureDevice

Part of a series of CLs for FakeVideoCaptureDevice with the goal of allowing us
to expand our test coverage of the video capture pipeline.

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL12.

BUG=584797
TEST=
  capture_unittests --gtest_filter="*Video*"

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

[chfremer, 2017-02-28 19:34:55]

chfremer@chromium.org changed reviewers:
+ emircan@chromium.org

[chfremer, 2017-02-28 19:34:55]

PTAL

[chfremer, 2017-02-28 19:35:01]

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

[commit-bot: I haz the power, 2017-02-28 19:35:47]

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

[mcasas, 2017-02-28 21:24:50]

On 2017/02/28 19:34:55, chfremer wrote:
> PTAL

Please make this CL look like e.g. https://crrev.com/2690253002/,
i.e.: make PS1 the original CL, PS2 the changes. If the code is 
exactly the same with just a rebase on ToT, then say so, otherwise
it's confusing to read 
"the same as in the original CL except for a one-line merge-in" 
without pointing at the exact line ;)
Otherwise lgtm.

[commit-bot: I haz the power, 2017-02-28 21:34:01]

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

[commit-bot: I haz the power, 2017-02-28 21:34:02]

Dry run: This issue passed the CQ dry run.

[emircan, 2017-02-28 21:48:42]

lgtm % what miguel said.

[chfremer, 2017-02-28 21:53:09]

Description was changed from

==========
Reland Add MJPEG support to FakeVideoCaptureDevice

This was reverted as suspected to cause some ChromeOS Autotest failures, see 
crbug.com/696130. Those failures turned out to be very likely caused by 
something else.

The code is the same as in the original CL except for a one-line merge-in from 
CL https://codereview.chromium.org/2673373003/.

Original CL description:

Add MJPEG support to FakeVideoCaptureDevice

Part of a series of CLs for FakeVideoCaptureDevice with the goal of allowing us
to expand our test coverage of the video capture pipeline.

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL12.

BUG=584797
TEST=
  capture_unittests --gtest_filter="*Video*"

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

to

==========
Reland Add MJPEG support to FakeVideoCaptureDevice

This was reverted as suspected to cause some ChromeOS Autotest failures, see 
crbug.com/696130. Those failures turned out to be very likely caused by 
something else.

The code is the same as in the original CL % rebase to ToT.

Original CL description:

Add MJPEG support to FakeVideoCaptureDevice

Part of a series of CLs for FakeVideoCaptureDevice with the goal of allowing us
to expand our test coverage of the video capture pipeline.

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL12.

BUG=584797
TEST=
  capture_unittests --gtest_filter="*Video*"

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

[chfremer, 2017-02-28 21:54:20]

On 2017/02/28 21:24:50, mcasas wrote:
> On 2017/02/28 19:34:55, chfremer wrote:
> > PTAL
> 
> Please make this CL look like e.g. https://crrev.com/2690253002/,
> i.e.: make PS1 the original CL, PS2 the changes. If the code is 
> exactly the same with just a rebase on ToT, then say so, otherwise
> it's confusing to read 
> "the same as in the original CL except for a one-line merge-in" 
> without pointing at the exact line ;)
> Otherwise lgtm.

Sorry about the confusing description.
To clarify, the code is exactly the same with just a rebase on ToT.

I updated the description to reflect this.

[chfremer, 2017-02-28 21:54:28]

The CQ bit was checked by chfremer@chromium.org

[commit-bot: I haz the power, 2017-02-28 21:55:01]

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, 2017-02-28 22:18:47]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488318868403740, "parent_rev":
"af45f47f94de96e49d4e93ad2dfbe806b9a3e3b0", "commit_rev":
"fe4c57f8d526e1c286d0504f40e5b178cc527e6c"}

[commit-bot: I haz the power, 2017-02-28 22:19:25]

Description was changed from

==========
Reland Add MJPEG support to FakeVideoCaptureDevice

This was reverted as suspected to cause some ChromeOS Autotest failures, see 
crbug.com/696130. Those failures turned out to be very likely caused by 
something else.

The code is the same as in the original CL % rebase to ToT.

Original CL description:

Add MJPEG support to FakeVideoCaptureDevice

Part of a series of CLs for FakeVideoCaptureDevice with the goal of allowing us
to expand our test coverage of the video capture pipeline.

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL12.

BUG=584797
TEST=
  capture_unittests --gtest_filter="*Video*"

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...
==========

to

==========
Reland Add MJPEG support to FakeVideoCaptureDevice

This was reverted as suspected to cause some ChromeOS Autotest failures, see 
crbug.com/696130. Those failures turned out to be very likely caused by 
something else.

The code is the same as in the original CL % rebase to ToT.

Original CL description:

Add MJPEG support to FakeVideoCaptureDevice

Part of a series of CLs for FakeVideoCaptureDevice with the goal of allowing us
to expand our test coverage of the video capture pipeline.

This CL is part of the Mojo Video Capture work. For the bigger picture,
see [1] CL12.

BUG=584797
TEST=
  capture_unittests --gtest_filter="*Video*"

[1]
https://docs.google.com/a/chromium.org/document/d/1Qw7rw1AJy0QHXjha36jZNiEuxs...

Review-Url: https://codereview.chromium.org/2721633006
Cr-Commit-Position: refs/heads/master@{#453726}
Committed:
https://chromium.googlesource.com/chromium/src/+/fe4c57f8d526e1c286d0504f40e5...
==========

[commit-bot: I haz the power, 2017-02-28 22:19:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/fe4c57f8d526e1c286d0504f40e5...