bug 1471953 use SystemGroup for ThreadedDriver thread shutdown task r=padenot
authorKarl Tomlinson <karlt+@karlt.net>
Tue, 03 Jul 2018 14:26:21 +1200
changeset 425510 3da753ca1fb121f8a17a52fde9136773c8e0a270
parent 425474 ae04f1ff217f85049f1d157517241274d15ab5bb
child 425511 a197a960866b4c074633f82d4984d792d4d6a1df
push id34256
push useraciure@mozilla.com
push dateMon, 09 Jul 2018 21:59:26 +0000
treeherdermozilla-central@19edc7c22303 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspadenot
bugs1471953
milestone63.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
bug 1471953 use SystemGroup for ThreadedDriver thread shutdown task r=padenot The shutdown of the thread is not observable from content, and so its order wrt other tasks is not important.
dom/media/GraphDriver.cpp
--- a/dom/media/GraphDriver.cpp
+++ b/dom/media/GraphDriver.cpp
@@ -174,17 +174,17 @@ private:
   nsCOMPtr<nsIThread> mThread;
 };
 
 ThreadedDriver::~ThreadedDriver()
 {
   if (mThread) {
     nsCOMPtr<nsIRunnable> event =
       new MediaStreamGraphShutdownThreadRunnable(mThread.forget());
-    GraphImpl()->Dispatch(event.forget());
+    SystemGroup::Dispatch(TaskCategory::Other, event.forget());
   }
 }
 
 class MediaStreamGraphInitThreadRunnable : public Runnable {
 public:
   explicit MediaStreamGraphInitThreadRunnable(ThreadedDriver* aDriver)
     : Runnable("MediaStreamGraphInitThreadRunnable")
     , mDriver(aDriver)