Bug 1309162 - part4 : remove checking for mPlayingBeforeSeek. r=baku
☠☠ backed out by eb95159069bb ☠ ☠
authorAlastor Wu <alwu@mozilla.com>
Sat, 26 Nov 2016 02:35:18 +0800
changeset 324527 fe1771c913099ef108b6f856cb7fa6e19bfc0c5f
parent 324526 b5ab5eae225d0dd31c4c2ce7de86a50094535bdd
child 324528 2015de6577af9b626dad5576e92120bccb59e719
push id24
push usermaklebus@msu.edu
push dateTue, 20 Dec 2016 03:11:33 +0000
reviewersbaku
bugs1309162
milestone53.0a1
Bug 1309162 - part4 : remove checking for mPlayingBeforeSeek. r=baku 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
@@ -925,21 +925,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;
   }