Bug 1309162 - part4 : remove checking for mPlayingBeforeSeek. r=baku
authorAlastor Wu <alwu@mozilla.com>
Tue, 29 Nov 2016 12:40:41 +0800
changeset 324591 67431dae9ec80e2d1651e1ae1baca1423462cb08
parent 324590 37f5c9c13c2feccde0150410fce8102b4dbafe0f
child 324592 583e3edec539669eb7d6c193d25f721c94328e0e
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;
   }