Bug 1309162 - part4 : remove checking for mPlayingBeforeSeek. draft
authorAlastor Wu <alwu@mozilla.com>
Thu, 17 Nov 2016 15:26:58 +0800
changeset 440274 9d987ec247cd0d01d920045f6e8740332c49b7f9
parent 440273 af1d3b71f5942300b2b075d757d85dbe515cee71
child 440275 9bbca288697fd3b7901970011a9f82ff772017ec
push id36191
push useralwu@mozilla.com
push dateThu, 17 Nov 2016 07:27:36 +0000
bugs1309162
milestone52.0a1
Bug 1309162 - part4 : remove checking for mPlayingBeforeSeek. In previous patch, we removed UpdateAudioChannelPlayingState() from AddRemoveSelfReference(), now we don't call UpdateAudioChannelPlayingState() during the seeking. So we don't need to afraid to stop audio channel agent during the seeking. MozReview-Commit-ID: GTuIRB24zlq
dom/html/HTMLMediaElement.cpp
--- a/dom/html/HTMLMediaElement.cpp
+++ b/dom/html/HTMLMediaElement.cpp
@@ -917,21 +917,16 @@ private:
       return true;
     }
 
     // If we are actually playing...
     if (mOwner->IsCurrentlyPlaying()) {
       return true;
     }
 
-    // If we are seeking, we consider it as playing
-    if (mOwner->mPlayingBeforeSeek) {
-      return true;
-    }
-
     // If we are playing an external stream.
     if (mOwner->mSrcAttrStream) {
       return true;
     }
 
     return false;
   }