Bug 1415556 - P12. Remove unecessary locking. r?padenot draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Sat, 11 Nov 2017 10:20:59 +0100
changeset 696759 9f6cd52fc7f02ef2e124c982789a10e223f0fcb7
parent 696758 c9099f4416c635d6128a830dac1b43cad4da27e3
child 739918 cc922ddd8ed4fba43bd14a06085dc2ad32c62d06
push id88781
push userbmo:jyavenard@mozilla.com
push dateSat, 11 Nov 2017 09:23:59 +0000
reviewerspadenot
bugs1415556
milestone58.0a1
Bug 1415556 - P12. Remove unecessary locking. r?padenot The method doesn't use any MSG member, only dispatching a task. MozReview-Commit-ID: 7uZbTvq9OQt
dom/media/MediaStreamGraph.cpp
--- a/dom/media/MediaStreamGraph.cpp
+++ b/dom/media/MediaStreamGraph.cpp
@@ -3986,18 +3986,16 @@ MediaStreamGraphImpl::SuspendOrResumeStr
 void
 MediaStreamGraphImpl::AudioContextOperationCompleted(MediaStream* aStream,
                                                      void* aPromise,
                                                      AudioContextOperation aOperation)
 {
   // This can be called from the thread created to do cubeb operation, or the
   // MSG thread. The pointers passed back here are refcounted, so are still
   // alive.
-  MonitorAutoLock lock(mMonitor);
-
   AudioContextState state;
   switch (aOperation) {
     case AudioContextOperation::Suspend:
       state = AudioContextState::Suspended;
       break;
     case AudioContextOperation::Resume:
       state = AudioContextState::Running;
       break;