media: Implement MojoVideoDecoder::CanReadWithoutStalling().

media: Implement MojoVideoDecoder::CanReadWithoutStalling().

This implementation is quite conservative; it only allows changes at the
same time as outputting a VideoFrame. This is fine, since the value is
irrelevant before the first frame, and the only dynamic case
(GpuVideoDecoder) changes values exactly on output.

BUG=522298
Review-Url: https://codereview.chromium.org/2951233002
Cr-Commit-Position: refs/heads/master@{#481989}
Committed: https://chromium.googlesource.com/chromium/src/+/7e2b9f8c41a053a3ca06552808a247e576258593

[sandersd (OOO until July 31), 2017-06-22 00:47:52]

sandersd@chromium.org changed reviewers:
+ watk@chromium.org

[sandersd (OOO until July 31), 2017-06-22 00:50:37]

Patchset #1 (id:1) has been deleted

[watk, 2017-06-23 00:23:44]

lgtm

[sandersd (OOO until July 31), 2017-06-23 00:33:36]

Description was changed from

==========
media: Implement MojoVideoDecoder::CanReadWithoutStalling().

This implementation is quite conservative; it only allows changes at the
same time as outputting a VideoFrame. This is fine, since the value is
irrelevant before the first frame, and the only dynamic case
(GpuVideoDecoder) changes values exactly on output.

BUG=522298
==========

to

==========
media: Implement MojoVideoDecoder::CanReadWithoutStalling().

This implementation is quite conservative; it only allows changes at the
same time as outputting a VideoFrame. This is fine, since the value is
irrelevant before the first frame, and the only dynamic case
(GpuVideoDecoder) changes values exactly on output.

BUG=522298
==========

[sandersd (OOO until July 31), 2017-06-23 00:33:36]

sandersd@chromium.org changed reviewers:
+ dcheng@chromium.org

[sandersd (OOO until July 31), 2017-06-23 00:33:53]

dcheng@: Please review changes in video_decoder.mojom.

[dcheng, 2017-06-23 08:11:55]

lgtm

[sandersd (OOO until July 31), 2017-06-23 17:37:26]

The CQ bit was checked by sandersd@chromium.org

[commit-bot: I haz the power, 2017-06-23 17:37:45]

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-06-23 19:31:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498239446427330,
"parent_rev": "df9ed4d64484384625320f308285a68bd79d65a8", "commit_rev":
"7e2b9f8c41a053a3ca06552808a247e576258593"}

[commit-bot: I haz the power, 2017-06-23 19:31:24]

Description was changed from

==========
media: Implement MojoVideoDecoder::CanReadWithoutStalling().

This implementation is quite conservative; it only allows changes at the
same time as outputting a VideoFrame. This is fine, since the value is
irrelevant before the first frame, and the only dynamic case
(GpuVideoDecoder) changes values exactly on output.

BUG=522298
==========

to

==========
media: Implement MojoVideoDecoder::CanReadWithoutStalling().

This implementation is quite conservative; it only allows changes at the
same time as outputting a VideoFrame. This is fine, since the value is
irrelevant before the first frame, and the only dynamic case
(GpuVideoDecoder) changes values exactly on output.

BUG=522298

Review-Url: https://codereview.chromium.org/2951233002
Cr-Commit-Position: refs/heads/master@{#481989}
Committed:
https://chromium.googlesource.com/chromium/src/+/7e2b9f8c41a053a3ca06552808a2...
==========

[commit-bot: I haz the power, 2017-06-23 19:31:25]

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