Bug 1295023 - ignore to set the same value for the ready state. draft
authorAlastor Wu <alwu@mozilla.com>
Thu, 18 Aug 2016 19:10:31 +0800
changeset 402529 c68afe22f2237bba5b8ca31e95796e0c4a83a140
parent 401500 fe895421dfbe1f1f8f1fc6a39bb20774423a6d74
child 528698 0308d259984573f06cd326a60cf58a41e92db084
push id26687
push useralwu@mozilla.com
push dateThu, 18 Aug 2016 11:10:52 +0000
bugs1295023
milestone51.0a1
Bug 1295023 - ignore to set the same value for the ready state. MozReview-Commit-ID: 9OWJfnWAwj9
dom/html/HTMLTrackElement.cpp
--- a/dom/html/HTMLTrackElement.cpp
+++ b/dom/html/HTMLTrackElement.cpp
@@ -387,16 +387,20 @@ HTMLTrackElement::ReadyState() const
   }
 
   return mTrack->ReadyState();
 }
 
 void
 HTMLTrackElement::SetReadyState(uint16_t aReadyState)
 {
+  if (ReadyState() == aReadyState) {
+    return;
+  }
+
   if (mTrack) {
     switch (aReadyState) {
       case TextTrackReadyState::Loaded:
         DispatchTrackRunnable(NS_LITERAL_STRING("load"));
         break;
       case TextTrackReadyState::FailedToLoad:
         DispatchTrackRunnable(NS_LITERAL_STRING("error"));
         break;