Bug 1100803 - Re-create a decoded media stream if necessary. r=roc

mozilla-spidermonkey · Dec 15, 2014 · 82666af · 82666af
1 parent 1973c34
commit 82666af
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/dom/media/MediaDecoder.cpp b/dom/media/MediaDecoder.cpp
@@ -266,6 +266,14 @@ MediaDecoder::DecodedStreamGraphListener::NotifyEvent(MediaStreamGraph* aGraph,
     aGraph->DispatchToMainThreadAfterStreamStateUpdate(event.forget());
   }
 }
+
+void MediaDecoder::RecreateDecodedStreamIfNecessary(int64_t aStartTimeUSecs)
+{
+  if (mInitialAudioCaptured) {
+    RecreateDecodedStream(aStartTimeUSecs);
+  }
+}
+
 void MediaDecoder::DestroyDecodedStream()
 {
   MOZ_ASSERT(NS_IsMainThread());

diff --git a/dom/media/MediaDecoder.h b/dom/media/MediaDecoder.h
@@ -851,6 +851,9 @@ class MediaDecoder : public nsIObserver,
   // The decoder monitor must be held.
   bool IsLogicallyPlaying();
 
+  // Re-create a decoded stream if audio being captured
+  void RecreateDecodedStreamIfNecessary(int64_t aStartTimeUSecs);
+
 #ifdef MOZ_EME
   // This takes the decoder monitor.
   virtual nsresult SetCDMProxy(CDMProxy* aProxy) MOZ_OVERRIDE;

diff --git a/dom/media/MediaDecoderStateMachine.cpp b/dom/media/MediaDecoderStateMachine.cpp
@@ -1624,9 +1624,7 @@ MediaDecoderStateMachine::StartSeek(const SeekTarget& aTarget)
 
   DECODER_LOG("Changed state to SEEKING (to %lld)", mSeekTarget.mTime);
   SetState(DECODER_STATE_SEEKING);
-  if (mDecoder->GetDecodedStream()) {
-    mDecoder->RecreateDecodedStream(seekTime - mStartTime);
-  }
+  mDecoder->RecreateDecodedStreamIfNecessary(seekTime - mStartTime);
   ScheduleStateMachine();
 }