tor-browser

commit 2d3fc94ccc137213e93b72b4af651538e6a1d009
parent fdbd7e3635fed1112c1209c9bcb345322cfb4360
Author: Jens Stutte <jstutte@mozilla.com>
Date:   Thu,  2 Oct 2025 10:10:42 +0000

Bug 1991836 - Avoid double registration of AMTelemetry observers in test_blocklist_statechange_telemetry.update_softblocked_to_hardblocked. r=rpl

Differential Revision: https://phabricator.services.mozilla.com/D266960

Diffstat:
M
toolkit/mozapps/extensions/AddonManager.sys.mjs
 | 
1
+
M
toolkit/mozapps/extensions/test/xpcshell/rs-blocklist/test_blocklist_statechange_telemetry.js
 | 
2
+-

2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/toolkit/mozapps/extensions/AddonManager.sys.mjs b/toolkit/mozapps/extensions/AddonManager.sys.mjs
@@ -5342,6 +5342,7 @@ AMTelemetry = {
 
   // NOTE: used by AddonTestUtils.promiseShutdownManager to ensure
   // we can initialize it again in AddonTestUtils.promiseStartupManager.
+  // Also used by test_blocklist_statechange_telemetry.js.
   async uninit() {
     if (!this.telemetrySetupDone) {
       return;
diff --git a/toolkit/mozapps/extensions/test/xpcshell/rs-blocklist/test_blocklist_statechange_telemetry.js b/toolkit/mozapps/extensions/test/xpcshell/rs-blocklist/test_blocklist_statechange_telemetry.js
@@ -748,7 +748,7 @@ add_task(async function update_softblocked_to_unblocked() {
 add_task(async function update_softblocked_to_hardblocked() {
   // Re-initialize AMTelemetry to make sure it is listening to the AOM
   // addon events onEnabled.
-  AMTelemetry.telemetrySetupDone = false;
+  await AMTelemetry.uninit();
   AMTelemetry.onStartup();
 
   Services.fog.testResetFOG();