Bug 1805820 - Hold the outer lock when notifying outerMonitor in Time…

…rFiresBeforeRunnableRuns test, r=emilio Differential Revision: https://phabricator.services.mozilla.com/D176076
dothq · Apr 21, 2023 · b602da3 · b602da3
1 parent 9345673
commit b602da3
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/xpcom/tests/gtest/TestDelayedRunnable.cpp b/xpcom/tests/gtest/TestDelayedRunnable.cpp
@@ -151,6 +151,7 @@ TEST(DelayedRunnable, TimerFiresBeforeRunnableRuns)
         // we leave scope.
         innerMonitor.Wait();
         // Notify the outer monitor that we've finished the async steps.
+        MonitorAutoLock outerLock(outerMonitor);
         outerMonitor.NotifyAll();
       })));
   // Wait for async steps before wrapping up the test case.