mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-21 01:05:45 +00:00
Bug 1381748 - Cleanup FetchConsumer workflow - part 2 - cleanup RegisterWorkerHolder, r=catalinb
This commit is contained in:
parent
cad3905562
commit
e3222ef75d
@ -343,7 +343,7 @@ FetchBodyConsumer<Derived>::Create(nsIGlobalObject* aGlobal,
|
||||
|
||||
if (!NS_IsMainThread()) {
|
||||
MOZ_ASSERT(workerPrivate);
|
||||
if (NS_WARN_IF(!consumer->RegisterWorkerHolder(workerPrivate))) {
|
||||
if (NS_WARN_IF(!consumer->RegisterWorkerHolder())) {
|
||||
aRv.Throw(NS_ERROR_FAILURE);
|
||||
return nullptr;
|
||||
}
|
||||
@ -451,15 +451,15 @@ FetchBodyConsumer<Derived>::AssertIsOnTargetThread() const
|
||||
|
||||
template <class Derived>
|
||||
bool
|
||||
FetchBodyConsumer<Derived>::RegisterWorkerHolder(WorkerPrivate* aWorkerPrivate)
|
||||
FetchBodyConsumer<Derived>::RegisterWorkerHolder()
|
||||
{
|
||||
MOZ_ASSERT(aWorkerPrivate);
|
||||
aWorkerPrivate->AssertIsOnWorkerThread();
|
||||
MOZ_ASSERT(mWorkerPrivate);
|
||||
mWorkerPrivate->AssertIsOnWorkerThread();
|
||||
|
||||
MOZ_ASSERT(!mWorkerHolder);
|
||||
mWorkerHolder.reset(new FetchBodyWorkerHolder<Derived>(this));
|
||||
|
||||
if (!mWorkerHolder->HoldWorker(aWorkerPrivate, Closing)) {
|
||||
if (!mWorkerHolder->HoldWorker(mWorkerPrivate, Closing)) {
|
||||
NS_WARNING("Failed to add workerHolder");
|
||||
mWorkerHolder = nullptr;
|
||||
return false;
|
||||
|
@ -86,7 +86,7 @@ private:
|
||||
AssertIsOnTargetThread() const;
|
||||
|
||||
bool
|
||||
RegisterWorkerHolder(workers::WorkerPrivate* aWorkerPrivate);
|
||||
RegisterWorkerHolder();
|
||||
|
||||
nsCOMPtr<nsIThread> mTargetThread;
|
||||
nsCOMPtr<nsIEventTarget> mMainThreadEventTarget;
|
||||
|
Loading…
Reference in New Issue
Block a user