From b359db62fddfa49e5aec93023777cc39634ece7b Mon Sep 17 00:00:00 2001 From: Jan Varga Date: Fri, 8 Feb 2019 21:02:19 +0100 Subject: [PATCH] Bug 1517089 - Part 15: Remove a hack for JetPack and DevTools fron QuotaManager:GetInfoFromPrincipal; r=asuth The special case is now handled in GetSpecialBaseDomain in ContentPrincipal.cpp Differential Revision: https://phabricator.services.mozilla.com/D19212 --- dom/quota/ActorsParent.cpp | 23 +++-------------------- 1 file changed, 3 insertions(+), 20 deletions(-) diff --git a/dom/quota/ActorsParent.cpp b/dom/quota/ActorsParent.cpp index a5bef68e95fd..8321dba03f13 100644 --- a/dom/quota/ActorsParent.cpp +++ b/dom/quota/ActorsParent.cpp @@ -5494,28 +5494,11 @@ nsresult QuotaManager::GetInfoFromPrincipal(nsIPrincipal* aPrincipal, if (aGroup) { nsCString baseDomain; rv = aPrincipal->GetBaseDomain(baseDomain); - if (NS_FAILED(rv)) { - // A hack for JetPack. - - nsCOMPtr uri; - rv = aPrincipal->GetURI(getter_AddRefs(uri)); - NS_ENSURE_SUCCESS(rv, rv); - - bool isIndexedDBURI = false; - rv = uri->SchemeIs("indexedDB", &isIndexedDBURI); - NS_ENSURE_SUCCESS(rv, rv); - - if (isIndexedDBURI) { - rv = NS_OK; - } - } NS_ENSURE_SUCCESS(rv, rv); - if (baseDomain.IsEmpty()) { - aGroup->Assign(origin); - } else { - aGroup->Assign(baseDomain + suffix); - } + MOZ_ASSERT(!baseDomain.IsEmpty()); + + aGroup->Assign(baseDomain + suffix); } if (aOrigin) {