From 000a5ee7687439f0609627b2637220ae87e45625 Mon Sep 17 00:00:00 2001 From: Ben Kelly Date: Thu, 21 Jan 2016 08:49:00 -0500 Subject: [PATCH] Bug 1241531 - Part 2: Move Cancel() to ServiceWorkerJob base class. r=ehsan --- dom/workers/ServiceWorkerManager.cpp | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/dom/workers/ServiceWorkerManager.cpp b/dom/workers/ServiceWorkerManager.cpp index 673ec4e53c10..69dc50749da7 100644 --- a/dom/workers/ServiceWorkerManager.cpp +++ b/dom/workers/ServiceWorkerManager.cpp @@ -168,6 +168,13 @@ public: virtual void Start() = 0; + void + Cancel() + { + mQueue = nullptr; + mCanceled = true; + } + bool IsRegisterOrInstallJob() const { @@ -181,10 +188,12 @@ protected: ServiceWorkerJobQueue* mQueue; Type mJobType; + bool mCanceled; explicit ServiceWorkerJob(ServiceWorkerJobQueue* aQueue, Type aJobType) : mQueue(aQueue) , mJobType(aJobType) + , mCanceled(false) {} virtual ~ServiceWorkerJob() @@ -952,19 +961,11 @@ public: , mScriptSpec(aScriptSpec) , mCallback(aCallback) , mUpdateAndInstallInfo(aServiceWorkerInfo) - , mCanceled(false) { AssertIsOnMainThread(); MOZ_ASSERT(aPrincipal); } - void - Cancel() - { - mQueue = nullptr; - mCanceled = true; - } - protected: nsCOMPtr mPrincipal; const nsCString mScope; @@ -972,7 +973,6 @@ protected: RefPtr mCallback; RefPtr mRegistration; RefPtr mUpdateAndInstallInfo; - bool mCanceled; ~ServiceWorkerJobBase() { }