From 7d60f7e735d0a363ce7043bcd973b7cc307ff4dc Mon Sep 17 00:00:00 2001 From: Greg Stoll Date: Wed, 20 Mar 2024 17:35:26 +0000 Subject: [PATCH] Bug 1882603 - don't show Content Analysis Block result if pref is set r=dlp-reviewers,handyman Differential Revision: https://phabricator.services.mozilla.com/D205153 --- .../contentanalysis/content/ContentAnalysis.sys.mjs | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/browser/components/contentanalysis/content/ContentAnalysis.sys.mjs b/browser/components/contentanalysis/content/ContentAnalysis.sys.mjs index f3d466aaa93e..dd5c57c11868 100644 --- a/browser/components/contentanalysis/content/ContentAnalysis.sys.mjs +++ b/browser/components/contentanalysis/content/ContentAnalysis.sys.mjs @@ -42,6 +42,13 @@ XPCOMUtils.defineLazyPreferenceGetter( "A DLP agent" ); +XPCOMUtils.defineLazyPreferenceGetter( + lazy, + "showBlockedResult", + "browser.contentanalysis.show_blocked_result", + true +); + /** * A class that groups browsing contexts by their top-level one. * This is necessary because if there may be a subframe that @@ -705,6 +712,10 @@ export const ContentAnalysis = { lazy.gContentAnalysis.respondToWarnDialog(aRequestToken, allow); return null; case Ci.nsIContentAnalysisResponse.eBlock: + if (!lazy.showBlockedResult) { + // Don't show anything + return null; + } message = await this.l10n.formatValue("contentanalysis-block-message", { content: this._getResourceNameFromNameOrOperationType( aResourceNameOrOperationType