Bug 1265969 - Don't fire pop events in MediaQueue::Reset(). r=jya. draft
authorJW Wang <jwwang@mozilla.com>
Tue, 19 Apr 2016 23:04:02 +0800
changeset 354178 e5b89db004f707c8aa24b00242791f7ec6039bc7
parent 354177 7570e8474f76a57bcf6718c2cf4423b90a596aa5
child 518916 fda531468e3f751998eca5554def343d2dfc7e8d
push id15983
push userjwwang@mozilla.com
push dateWed, 20 Apr 2016 06:16:39 +0000
reviewersjya
bugs1265969
milestone48.0a1
Bug 1265969 - Don't fire pop events in MediaQueue::Reset(). r=jya. MozReview-Commit-ID: 6J0VsVvIeg
dom/media/MediaQueue.h
--- a/dom/media/MediaQueue.h
+++ b/dom/media/MediaQueue.h
@@ -74,17 +74,17 @@ public:
   inline T* PeekFront() {
     ReentrantMonitorAutoEnter mon(mReentrantMonitor);
     return static_cast<T*>(nsDeque::PeekFront());
   }
 
   void Reset() {
     ReentrantMonitorAutoEnter mon(mReentrantMonitor);
     while (GetSize() > 0) {
-      RefPtr<T> x = PopFront();
+      RefPtr<T> x = dont_AddRef(static_cast<T*>(nsDeque::PopFront()));
     }
     mEndOfStream = false;
   }
 
   bool AtEndOfStream() {
     ReentrantMonitorAutoEnter mon(mReentrantMonitor);
     return GetSize() == 0 && mEndOfStream;
   }