Update the WebMediaSourceClientImpl & ChunkDemuxer to received explicit end of stream cancellation.

Update the WebMediaSourceClientImpl & ChunkDemuxer to received explicit end of stream cancellation.

There are other MediaSource API interactions other than append() that can cause a transition
out of the "end of stream" state. This patch simply makes it possible for ChunkDemuxer to
get explicitly notified by Blink when entering/leaving the "end of stream" state.

BUG=239506
TEST=All existing tests pass. Additional LayoutTests will be added to cover non-append related
     transitions out of the "end of stream" state.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=212753

[acolwell GONE FROM CHROMIUM, 2013-07-19 20:27:08]

FYI. The mark/unmark convention came from abarth@ on the Blink-side changes. I'm
just propagating them here so all the code is consistent.

[scherkus (not reviewing), 2013-07-19 20:35:47]

is it legal to call unmark if it hasn't been marked?

https://codereview.chromium.org/19649007/diff/15001/content/renderer/media/we...
File content/renderer/media/webmediasourceclient_impl.cc (right):

https://codereview.chromium.org/19649007/diff/15001/content/renderer/media/we...
content/renderer/media/webmediasourceclient_impl.cc:86: 
remove extra blank

https://codereview.chromium.org/19649007/diff/15001/media/filters/chunk_demux...
File media/filters/chunk_demuxer.h (right):

https://codereview.chromium.org/19649007/diff/15001/media/filters/chunk_demux...
media/filters/chunk_demuxer.h:127: // Signals an EndOfStream request.
do you want to keep the EndOfStream formatting in this comment?

[acolwell GONE FROM CHROMIUM, 2013-07-19 20:45:54]

On 2013/07/19 20:35:47, scherkus wrote:
> is it legal to call unmark if it hasn't been marked?
No. That is why I have the DCHECK_EQ(state_, ENDED) in
ChunkDemuxer::UnmarkEndOfStream(). I added a sentence in the header file to
clarify this as well.

https://codereview.chromium.org/19649007/diff/15001/content/renderer/media/we...
File content/renderer/media/webmediasourceclient_impl.cc (right):

https://codereview.chromium.org/19649007/diff/15001/content/renderer/media/we...
content/renderer/media/webmediasourceclient_impl.cc:86: 
On 2013/07/19 20:35:47, scherkus wrote:
> remove extra blank

Done.

https://codereview.chromium.org/19649007/diff/15001/media/filters/chunk_demux...
File media/filters/chunk_demuxer.h (right):

https://codereview.chromium.org/19649007/diff/15001/media/filters/chunk_demux...
media/filters/chunk_demuxer.h:127: // Signals an EndOfStream request.
On 2013/07/19 20:35:47, scherkus wrote:
> do you want to keep the EndOfStream formatting in this comment?

Nope. Reworded the comment.

[scherkus (not reviewing), 2013-07-19 21:23:42]

lgtm

[commit-bot: I haz the power, 2013-07-19 21:28:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/acolwell@chromium.org/19649007/21001

[commit-bot: I haz the power, 2013-07-19 21:50:51]

Sorry for I got bad news for ya.
Compile failed with a clobber build on android_clang_dbg.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-07-19 22:21:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/acolwell@chromium.org/19649007/36001

[commit-bot: I haz the power, 2013-07-19 22:36:49]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

[commit-bot: I haz the power, 2013-07-19 23:14:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/acolwell@chromium.org/19649007/36001

[commit-bot: I haz the power, 2013-07-20 05:48:08]

Change committed as 212753