Bug 1265974 - Don't fire push events in MediaQueue::PushFront(). r=jya. draft
authorJW Wang <jwwang@mozilla.com>
Thu, 21 Apr 2016 10:18:20 +0800
changeset 354526 b3bdf588259be34078fe2284ac339c0100c283f5
parent 354481 4feb4dd910a5a2d3061dbdd376a80975206819c6
child 355210 4ec55f5662fe4747399aa45ab486dcdb23d085d1
push id16109
push userjwwang@mozilla.com
push dateThu, 21 Apr 2016 03:19:57 +0000
reviewersjya
bugs1265974
milestone48.0a1
Bug 1265974 - Don't fire push events in MediaQueue::PushFront(). r=jya. MozReview-Commit-ID: FEqDlD854vM
dom/media/MediaQueue.h
--- a/dom/media/MediaQueue.h
+++ b/dom/media/MediaQueue.h
@@ -49,17 +49,16 @@ public:
     mPushEvent.Notify(RefPtr<T>(aItem));
   }
 
   inline void PushFront(T* aItem) {
     ReentrantMonitorAutoEnter mon(mReentrantMonitor);
     MOZ_ASSERT(aItem);
     NS_ADDREF(aItem);
     nsDeque::PushFront(aItem);
-    mPushEvent.Notify(RefPtr<T>(aItem));
   }
 
   inline already_AddRefed<T> PopFront() {
     ReentrantMonitorAutoEnter mon(mReentrantMonitor);
     RefPtr<T> rv = dont_AddRef(static_cast<T*>(nsDeque::PopFront()));
     if (rv) {
       mPopEvent.Notify(rv);
     }