From dca80bb57075308340a065b9059ae8afe23f9aa5 Mon Sep 17 00:00:00 2001 From: Andrew McCreight Date: Wed, 25 Sep 2019 22:40:49 +0000 Subject: [PATCH] Bug 1579835, part 5 - Fix remote type matching in SelectTargetActorForServiceWorker. r=asuth With Fission, the remote type can have the form webIsolated= in addition to web. IsWebRemoteType() handles both of these cases. Differential Revision: https://phabricator.services.mozilla.com/D45378 --HG-- extra : moz-landing-system : lando --- dom/workers/remoteworkers/RemoteWorkerManager.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dom/workers/remoteworkers/RemoteWorkerManager.cpp b/dom/workers/remoteworkers/RemoteWorkerManager.cpp index d50a7e9e473a..17c628509f74 100644 --- a/dom/workers/remoteworkers/RemoteWorkerManager.cpp +++ b/dom/workers/remoteworkers/RemoteWorkerManager.cpp @@ -226,7 +226,7 @@ RemoteWorkerManager::SelectTargetActorForServiceWorker() const { auto scopeExit = MakeScopeExit( [&] { contentParents.AppendElement(std::move(contentParent)); }); - if (contentParent->GetRemoteType().EqualsLiteral(DEFAULT_REMOTE_TYPE)) { + if (IsWebRemoteType(contentParent->GetRemoteType())) { auto lock = contentParent->mRemoteWorkerActorData.Lock(); if (lock->mCount || !lock->mShutdownStarted) {