Bug 1841610 - Remove chat message theme telemetry probe. r=darktrojan

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

--HG--
extra : amend_source : b75adb6c8a720d4fb83f6e231af816f1ae2be3b7

mail/components/im/content/chat-conversation.js

@@ -62,14 +62,6 @@
 
             Services.obs.removeObserver(this.observer, "conversation-loaded");
 
-            // Report the active chat message theme via telemetry. This is not
-            // inside the conv browser itself, since the browser is also used
-            // for the theme preview in the settings.
-            Services.telemetry.scalarSet(
-              "tb.chat.active_message_theme",
-              `${this.convBrowser.theme.name}:${this.convBrowser.theme.variant}`
-            );
-
             return;
           }
 

mail/components/im/test/browser/browser.ini

@@ -14,7 +14,6 @@ head = head.js
 
 [browser_browserRequest.js]
 [browser_chatNotifications.js]
-[browser_chatTelemetry.js]
 [browser_contextMenu.js]
 [browser_logs.js]
 [browser_messagesMail.js]

mail/components/im/test/browser/browser_chatTelemetry.js

@@ -1,54 +0,0 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
- * License, v. 2.0. If a copy of the MPL was not distributed with this
- * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-let { TelemetryTestUtils } = ChromeUtils.importESModule(
-  "resource://testing-common/TelemetryTestUtils.sys.mjs"
-);
-
-add_task(async function testMessageThemeTelemetry() {
-  Services.telemetry.clearScalars();
-
-  const account = IMServices.accounts.createAccount(
-    "testuser",
-    "prpl-mochitest"
-  );
-  let passwordPromise = TestUtils.topicObserved("account-updated");
-  account.password = "this is a test";
-  await passwordPromise;
-  account.connect();
-
-  let scalars = TelemetryTestUtils.getProcessScalars("parent");
-  ok(
-    !scalars["tb.chat.active_message_theme"],
-    "Active chat theme not reported without open conversation."
-  );
-
-  await openChatTab();
-  ok(BrowserTestUtils.is_visible(document.getElementById("chatPanel")));
-
-  const conversation = account.prplAccount.wrappedJSObject.makeDM("collapse");
-  const convNode = getConversationItem(conversation);
-  ok(convNode);
-
-  await EventUtils.synthesizeMouseAtCenter(convNode, {});
-
-  const chatConv = getChatConversationElement(conversation);
-  const conversationLoaded = waitForConversationLoad(chatConv.convBrowser);
-  ok(chatConv, "found conversation");
-  ok(BrowserTestUtils.is_visible(chatConv), "conversation visible");
-  await BrowserTestUtils.browserLoaded(chatConv.convBrowser);
-
-  await conversationLoaded;
-  scalars = TelemetryTestUtils.getProcessScalars("parent");
-  // NOTE: tb.chat.active_message_theme expires at v 117.
-  is(
-    scalars["tb.chat.active_message_theme"],
-    "mail:default",
-    "Active chat message theme and variant reported after opening conversation."
-  );
-
-  conversation.close();
-  account.disconnect();
-  IMServices.accounts.deleteAccount(account.id);
-});

mail/components/telemetry/Scalars.yaml

@@ -572,20 +572,3 @@ tb.ui.interaction:
     release_channel_collection: opt-out
     record_in_processes:
       - "main"
-
-tb.chat:
-  active_message_theme:
-    bug_numbers:
-      - 1767004
-    description: >
-      Records the currently active chat message theme and variant.
-    expires: "117"
-    kind: string
-    keyed: false
-    notification_emails:
-      - "telemetry-client-dev@thunderbird.net"
-    products:
-      - "thunderbird"
-    release_channel_collection: opt-out
-    record_in_processes:
-      - "main"