DescriptionFix various MediaSource related crashes on Android.
This change fixes several issues that were causing crashes.
- MediaPlayerHostMsg_DemuxerReady_Params was not initializing the xxx_codec
member variables so audio-only or video-only content would randomly be
flagged as A/V content with a random codec ID.
- ChunkDemuxer::Stop() was not being called which could cause the delegate
to be accessed after it was destroyed.
- MediaSourceDelegate was signalling that it had metadata before the
ChunkDemuxer was even initialized. This caused the HTMLMediaElement to
signal that the load completed way too early and caused play() to be
called when the delegate wasn't ready.
BUG=233420
TEST=webkitmediasource-play.html LayoutTest reliably runs w/o crashing now.
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=202019
Patch Set 1 : Original patch #Patch Set 2 : Make Destroy() public and remove friend decl to make compiler happy #
Messages
Total messages: 4 (0 generated)
|