Bug 1365504 - Remove the call to AbstractThread::GetCurrent() in ServiceWorkerUpdaterChild.cpp. r?bkelly draft
authorJW Wang <jwwang@mozilla.com>
Wed, 17 May 2017 14:35:00 +0800
changeset 579948 df285ffdc42d8359b31eeabdc15932c1dd8fe7e1
parent 579289 2550b12f5f267e06f35221bd18f4ffeec66461ae
child 580107 0692ac5a6aa2c8ea61e5464afef87f63b00c2d36
push id59420
push userjwwang@mozilla.com
push dateThu, 18 May 2017 02:54:08 +0000
reviewersbkelly
bugs1365504, 1365483
milestone55.0a1
Bug 1365504 - Remove the call to AbstractThread::GetCurrent() in ServiceWorkerUpdaterChild.cpp. r?bkelly See bug 1365483 for the rationale. MozReview-Commit-ID: BDbdiWT2ZFa
dom/workers/ServiceWorkerUpdaterChild.cpp
--- a/dom/workers/ServiceWorkerUpdaterChild.cpp
+++ b/dom/workers/ServiceWorkerUpdaterChild.cpp
@@ -11,21 +11,24 @@ namespace dom {
 namespace workers {
 
 ServiceWorkerUpdaterChild::ServiceWorkerUpdaterChild(GenericPromise* aPromise,
                                                      CancelableRunnable* aSuccessRunnable,
                                                      CancelableRunnable* aFailureRunnable)
   : mSuccessRunnable(aSuccessRunnable)
   , mFailureRunnable(aFailureRunnable)
 {
+  // TODO: remove the main thread restriction after fixing bug 1364821.
+  MOZ_ASSERT(NS_IsMainThread());
+
   MOZ_ASSERT(aPromise);
   MOZ_ASSERT(aSuccessRunnable);
   MOZ_ASSERT(aFailureRunnable);
 
-  aPromise->Then(AbstractThread::GetCurrent(), __func__,
+  aPromise->Then(AbstractThread::MainThread(), __func__,
     [this]() {
       mPromiseHolder.Complete();
       Unused << Send__delete__(this);
   }).Track(mPromiseHolder);
 }
 
 mozilla::ipc::IPCResult
 ServiceWorkerUpdaterChild::RecvProceed(const bool& aAllowed)