Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Commit f8da7ca

Browse files
Alexandru Marcamarc-mozilla
authored andcommitted
Revert "Bug 1975403 - Telemetry for containers, r=tabbrowser-reviewers,chutten,sthompson" for causing bc failures @ browser_telemetry.js
This reverts commit ff922519e4ad70ae0e191f1303b004912c579bdf.
1 parent fe90849 commit f8da7ca

File tree

9 files changed

+0
-406
lines changed

9 files changed

+0
-406
lines changed

browser/components/contextualidentity/metrics.yaml

Lines changed: 0 additions & 161 deletions
This file was deleted.

browser/components/contextualidentity/test/browser/browser.toml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -78,13 +78,6 @@ support-files = [
7878

7979
["browser_tab_color_update.js"]
8080

81-
["browser_telemetry.js"]
82-
https_first_disabled = true
83-
skip-if = [
84-
"os == 'mac' && os_version == '11.20' && arch == 'aarch64'", # Bug 1783491
85-
"os == 'mac' && os_version == '15.30' && arch == 'aarch64'", # Bug 1783491
86-
]
87-
8881
["browser_usercontext.js"]
8982

9083
["browser_usercontextid_new_window.js"]

browser/components/contextualidentity/test/browser/browser_telemetry.js

Lines changed: 0 additions & 107 deletions
This file was deleted.

browser/components/tabbrowser/content/tabbrowser.js

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9448,11 +9448,6 @@ var TabContextMenu = {
94489448
triggeringPrincipal,
94499449
});
94509450

9451-
Glean.containers.tabAssignedContainer.record({
9452-
from_container_id: tab.getAttribute("usercontextid"),
9453-
to_container_id: userContextId,
9454-
});
9455-
94569451
if (gBrowser.selectedTab == tab) {
94579452
gBrowser.selectedTab = newTab;
94589453
}

browser/modules/BrowserUsageTelemetry.sys.mjs

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1263,13 +1263,6 @@ export let BrowserUsageTelemetry = {
12631263
Glean.tabgroup.tabInteractions.new.add();
12641264
}
12651265

1266-
const userContextId = event?.target?.getAttribute("usercontextid");
1267-
if (userContextId) {
1268-
Glean.containers.containerTabOpened.record({
1269-
container_id: String(userContextId),
1270-
});
1271-
}
1272-
12731266
// In the case of opening multiple tabs at once, avoid enumerating all open
12741267
// tabs and windows each time a tab opens.
12751268
this._onTabsOpenedTask.disarm();
@@ -1304,13 +1297,6 @@ export let BrowserUsageTelemetry = {
13041297
}
13051298
}
13061299

1307-
const userContextId = event?.target?.getAttribute("usercontextid");
1308-
if (userContextId) {
1309-
Glean.containers.containerTabClosed.record({
1310-
container_id: String(userContextId),
1311-
});
1312-
}
1313-
13141300
if (event.target?.pinned) {
13151301
const pinnedTabs = getPinnedTabsCount();
13161302
this.recordPinnedTabsCount(pinnedTabs - 1);

toolkit/components/contextualidentity/ContextualIdentityService.sys.mjs

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -159,10 +159,6 @@ _ContextualIdentityService.prototype = {
159159
this.notifyAllContainersCleared();
160160
this.resetDefault();
161161
}
162-
163-
Glean.containers.containersEnabled.record({
164-
enabled: contextualIdentitiesEnabled,
165-
});
166162
}
167163
},
168164

@@ -184,11 +180,6 @@ _ContextualIdentityService.prototype = {
184180
this.loadError(error);
185181
}
186182
);
187-
188-
Glean.containers.containerProfileLoaded.record({
189-
containers: this.getPublicUserContextIds().length,
190-
});
191-
192183
Services.obs.notifyObservers(
193184
null,
194185
"contextual-identity-service-load-finished"
@@ -306,10 +297,6 @@ _ContextualIdentityService.prototype = {
306297
"contextual-identity-created"
307298
);
308299

309-
Glean.containers.containerCreated.record({
310-
container_id: userContextId.toString(),
311-
});
312-
313300
return Cu.cloneInto(identity, {});
314301
},
315302

@@ -339,10 +326,6 @@ _ContextualIdentityService.prototype = {
339326
);
340327
}
341328

342-
Glean.containers.containerModified.record({
343-
container_id: userContextId.toString(),
344-
});
345-
346329
return !!identity;
347330
},
348331

@@ -398,10 +381,6 @@ _ContextualIdentityService.prototype = {
398381
this.saveSoon();
399382
Services.obs.notifyObservers(deletedOutput, "contextual-identity-deleted");
400383

401-
Glean.containers.containerDeleted.record({
402-
container_id: userContextId.toString(),
403-
});
404-
405384
return true;
406385
},
407386

0 commit comments

Comments
 (0)