Bug 1351874 - Dispatch shutdown runnable to CDMParent rather than null. r?gerald draft
authorChris Pearce <cpearce@mozilla.com>
Thu, 30 Mar 2017 11:03:50 +1300
changeset 553371 f1c5709acdefd4da002ff859aa9c3ef6fb275a4d
parent 552080 439653f6c5b332dd22fec23f6086ff1ede2f1be7
child 622044 ac37a1608cf53dc5dce1bc571991a1b7c525111d
push id51613
push userbmo:cpearce@mozilla.com
push dateWed, 29 Mar 2017 22:06:13 +0000
reviewersgerald
bugs1351874
milestone55.0a1
Bug 1351874 - Dispatch shutdown runnable to CDMParent rather than null. r?gerald MozReview-Commit-ID: BE5OgrgbvAT
dom/media/gmp/ChromiumCDMProxy.cpp
--- a/dom/media/gmp/ChromiumCDMProxy.cpp
+++ b/dom/media/gmp/ChromiumCDMProxy.cpp
@@ -351,17 +351,17 @@ ChromiumCDMProxy::Shutdown()
   mKeys.Clear();
   RefPtr<gmp::ChromiumCDMParent> cdm;
   {
     MutexAutoLock lock(mCDMMutex);
     cdm.swap(mCDM);
   }
   if (cdm) {
     nsCOMPtr<nsIRunnable> task =
-      NewRunnableMethod(mCDM, &gmp::ChromiumCDMParent::Shutdown);
+      NewRunnableMethod(cdm, &gmp::ChromiumCDMParent::Shutdown);
     mGMPThread->Dispatch(task.forget());
   }
 }
 
 void
 ChromiumCDMProxy::RejectPromise(PromiseId aId,
                                 nsresult aCode,
                                 const nsCString& aReason)