Bug 1316145 - Pass DontAssertDispatchSuccess when dispatching events because it is hard to enforce notification happens before AbstractThread shutdown. r=jya, a=jcristau
authorJW Wang <jwwang@mozilla.com>
Fri, 18 Nov 2016 17:50:27 +0800
changeset 352572 8bcdcb64549b43a1c6025ee4170c182567f70248
parent 352571 0d440483f80c9b8072b3aa6b55e91824c92f7f61
child 352573 239c6400a45d8e9a9a074befe45798fc9b2e95ff
push id6795
push userjlund@mozilla.com
push dateMon, 23 Jan 2017 14:19:46 +0000
treeherdermozilla-esr52@76101b503191 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjya, jcristau
bugs1316145
milestone52.0a2
Bug 1316145 - Pass DontAssertDispatchSuccess when dispatching events because it is hard to enforce notification happens before AbstractThread shutdown. r=jya, a=jcristau MozReview-Commit-ID: 2SATSQfdskl
dom/media/MediaEventSource.h
--- a/dom/media/MediaEventSource.h
+++ b/dom/media/MediaEventSource.h
@@ -106,17 +106,17 @@ struct EventTarget<DispatchPolicy::Async
     aTarget->Dispatch(Move(aTask), NS_DISPATCH_NORMAL);
   }
 };
 
 template <>
 struct EventTarget<DispatchPolicy::Async, AbstractThread> {
   static void
   Dispatch(AbstractThread* aTarget, already_AddRefed<nsIRunnable> aTask) {
-    aTarget->Dispatch(Move(aTask));
+    aTarget->Dispatch(Move(aTask), AbstractThread::DontAssertDispatchSuccess);
   }
 };
 
 template <>
 struct EventTarget<DispatchPolicy::Sync, nsIEventTarget> {
   static bool IsOnCurrentThread(nsIEventTarget* aTarget) {
     bool current = false;
     auto rv = aTarget->IsOnCurrentThread(&current);