Bug 1319706. Part 4 - add some logs to debug 'ended' not fired. r?kaku draft
authorJW Wang <jwwang@mozilla.com>
Thu, 24 Nov 2016 14:58:50 +0800
changeset 445186 234a7ec9d28778046bd19f9e218a19618a2e9b21
parent 445185 ec045344e92e0df172d98049a2b376a26cae560d
child 445321 f7a1dcc9982c0e0a0d008f5d5e74599c2fdc2a96
push id37455
push userjwwang@mozilla.com
push dateTue, 29 Nov 2016 09:40:57 +0000
reviewerskaku
bugs1319706
milestone53.0a1
Bug 1319706. Part 4 - add some logs to debug 'ended' not fired. r?kaku MozReview-Commit-ID: ATqszBBdo1P
dom/media/MediaDecoder.cpp
--- a/dom/media/MediaDecoder.cpp
+++ b/dom/media/MediaDecoder.cpp
@@ -980,19 +980,24 @@ MediaDecoder::IsShutdown() const
 
 void
 MediaDecoder::PlaybackEnded()
 {
   MOZ_ASSERT(NS_IsMainThread());
   MOZ_DIAGNOSTIC_ASSERT(!IsShutdown());
 
   if (mLogicallySeeking || mPlayState == PLAY_STATE_LOADING) {
+    DECODER_LOG("MediaDecoder::PlaybackEnded bailed out, "
+                "mLogicallySeeking=%d mPlayState=%s",
+                mLogicallySeeking.Ref(), ToPlayStateStr(mPlayState));
     return;
   }
 
+  DECODER_LOG("MediaDecoder::PlaybackEnded");
+
   ChangeState(PLAY_STATE_ENDED);
   InvalidateWithFlags(VideoFrameContainer::INVALIDATE_FORCE);
   mOwner->PlaybackEnded();
 
   // This must be called after |mOwner->PlaybackEnded()| call above, in order
   // to fire the required durationchange.
   if (IsInfinite()) {
     SetInfinite(false);