From 8d0fd9b1f8f4603123cfbc27c39ff0ede0056846 Mon Sep 17 00:00:00 2001 From: Tim Huang Date: Tue, 26 May 2020 07:27:25 +0000 Subject: [PATCH] Bug 1637226 - Part 3: Revert the assertion in ClientSource.cpp back to MOZ_DIAGNOSTIC_ASSERT. r=dimi Depends on D76732 Differential Revision: https://phabricator.services.mozilla.com/D76733 --- dom/clients/manager/ClientSource.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/dom/clients/manager/ClientSource.cpp b/dom/clients/manager/ClientSource.cpp index 67b33f9f8395..a7a7ffa1be8f 100644 --- a/dom/clients/manager/ClientSource.cpp +++ b/dom/clients/manager/ClientSource.cpp @@ -262,9 +262,10 @@ nsresult ClientSource::WindowExecutionReady(nsPIDOMWindowInner* aInnerWindow) { // continue to inherit the SW as well. We need to avoid triggering the // assertion in this corner case. if (mController.isSome()) { - MOZ_ASSERT(spec.LowerCaseEqualsLiteral("about:blank") || - StringBeginsWith(spec, NS_LITERAL_CSTRING("blob:")) || - StorageAllowedForWindow(aInnerWindow) == StorageAccess::eAllow); + MOZ_DIAGNOSTIC_ASSERT(spec.LowerCaseEqualsLiteral("about:blank") || + StringBeginsWith(spec, NS_LITERAL_CSTRING("blob:")) || + StorageAllowedForWindow(aInnerWindow) == + StorageAccess::eAllow); } nsPIDOMWindowOuter* outer = aInnerWindow->GetOuterWindow();