diff options
| author | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-23 14:41:34 +0000 |
|---|---|---|
| committer | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-23 14:41:34 +0000 |
| commit | 5e18d6ccc3fe668c6d47fbaafa6c603038d0be4c (patch) | |
| tree | eb013f8fdaaf4c44259f624a79e1053e89c1f25c /media/tools | |
| parent | 91a3e71461cfd94dbccc6c1e3d223962b43c284f (diff) | |
| download | chromium_src-5e18d6ccc3fe668c6d47fbaafa6c603038d0be4c.zip chromium_src-5e18d6ccc3fe668c6d47fbaafa6c603038d0be4c.tar.gz chromium_src-5e18d6ccc3fe668c6d47fbaafa6c603038d0be4c.tar.bz2 | |
Revert 201791 "Fix various MediaSource related crashes on Android."
> Fix 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.
>
> Review URL: https://chromiumcodereview.appspot.com/15754004
TBR=acolwell@chromium.org
Review URL: https://codereview.chromium.org/15838006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@201797 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/tools')
0 files changed, 0 insertions, 0 deletions