Bug 1286487 - WorkerProxyToMainThreadRunnable must keep alive workers using WorkerHolder - part 2. r=me, a=abillings
authorAndrea Marchesini <amarchesini@mozilla.com>
Tue, 16 Aug 2016 09:35:21 +0200
changeset 347716 d927ac40c9a6803c3338e5388d8222cada59a76f
parent 347715 4ea03e074e7f2f493b9b9a8ca979ee847094857b
child 347717 8ececbd9880dc48a879d7a0d44dbf5e15248b09c
push id6389
push userraliiev@mozilla.com
push dateMon, 19 Sep 2016 13:38:22 +0000
treeherdermozilla-beta@01d67bfe6c81 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme, abillings
bugs1286487
milestone50.0a2
Bug 1286487 - WorkerProxyToMainThreadRunnable must keep alive workers using WorkerHolder - part 2. r=me, a=abillings
dom/console/Console.cpp
--- a/dom/console/Console.cpp
+++ b/dom/console/Console.cpp
@@ -335,17 +335,18 @@ public:
     mWorkerPrivate->AssertIsOnWorkerThread();
 
     if (NS_WARN_IF(!PreDispatch(aCx))) {
       RunBackOnWorkerThread();
       return false;
     }
 
     if (NS_WARN_IF(!WorkerProxyToMainThreadRunnable::Dispatch())) {
-      RunBackOnWorkerThread();
+      // RunBackOnWorkerThread() will be called by
+      // WorkerProxyToMainThreadRunnable::Dispatch().
       return false;
     }
 
     return true;
   }
 
 protected:
   void