Bug 1309162 - part4 : remove checking for mPlayingBeforeSeek. draft
authorAlastor Wu <alwu@mozilla.com>
Tue, 29 Nov 2016 12:40:41 +0800
changeset 445136 9f72a36bee8b001daa3ba219cdb5b9b9b5600f62
parent 445135 c9d184cbc0638e45529492fcf858c3bdb4bf9163
child 445137 d0bb75fa673d31ccff4ed616dfc0cc099206fa05
push id37425
push useralwu@mozilla.com
push dateTue, 29 Nov 2016 05:50:50 +0000
bugs1309162
milestone53.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
@@ -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;
   }