media: Continue aborting FFMpegDemuxerStream::Read() calls until Seek().

media: Continue aborting FFMpegDemuxerStream::Read() calls until Seek().

For decoders that support multiple in-flight decodes (ie.
GpuVideoDecoder), it was possible to schedule a Read() after
AbortPendingReads() but before Seek() was called. These reads were
satisfied with data that decoders were not expecting (because the
aborted buffers were fully dropped, there were gaps in the input).

Now AbortPendingReads() is defined to continue to abort Read()s until
Seek(), which solves the problem. ChunkDemuxer was already implementing
this behavior, so only FFmpegDemuxer requires changes.

BUG=667210
Review-Url: https://codereview.chromium.org/2546263002
Cr-Commit-Position: refs/heads/master@{#436428}
(cherry picked from commit 216002cff0b287035a57cacc510419e15e5217c9)

TBR=dalecurtis@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/3131222425a01a6b46f7e19f47b938200fda807d

[sandersd (OOO until July 31), 2017-01-03 22:29:27]

Description was changed from

==========
media: Continue aborting FFMpegDemuxerStream::Read() calls until Seek().

For decoders that support multiple in-flight decodes (ie.
GpuVideoDecoder), it was possible to schedule a Read() after
AbortPendingReads() but before Seek() was called. These reads were
satisfied with data that decoders were not expecting (because the
aborted buffers were fully dropped, there were gaps in the input).

Now AbortPendingReads() is defined to continue to abort Read()s until
Seek(), which solves the problem. ChunkDemuxer was already implementing
this behavior, so only FFmpegDemuxer requires changes.

BUG=667210

Review-Url: https://codereview.chromium.org/2546263002
Cr-Commit-Position: refs/heads/master@{#436428}
(cherry picked from commit 216002cff0b287035a57cacc510419e15e5217c9)

TBR=dalecurtis@chromium.org
==========

to

==========
media: Continue aborting FFMpegDemuxerStream::Read() calls until Seek().

For decoders that support multiple in-flight decodes (ie.
GpuVideoDecoder), it was possible to schedule a Read() after
AbortPendingReads() but before Seek() was called. These reads were
satisfied with data that decoders were not expecting (because the
aborted buffers were fully dropped, there were gaps in the input).

Now AbortPendingReads() is defined to continue to abort Read()s until
Seek(), which solves the problem. ChunkDemuxer was already implementing
this behavior, so only FFmpegDemuxer requires changes.

BUG=667210

Review-Url: https://codereview.chromium.org/2546263002
Cr-Commit-Position: refs/heads/master@{#436428}
(cherry picked from commit 216002cff0b287035a57cacc510419e15e5217c9)

TBR=dalecurtis@chromium.org

Committed:
https://chromium.googlesource.com/chromium/src/+/3131222425a01a6b46f7e19f47b9...
==========

[sandersd (OOO until July 31), 2017-01-03 22:29:28]

Committed patchset #1 (id:1) manually as
3131222425a01a6b46f7e19f47b938200fda807d (presubmit successful).