Skip to content

Commit 9c2e34b

Browse files
ergunshDevtools-frontend LUCI CQ
authored andcommitted
[UserMetrics] Remove instrumenting PanelShownInLocation UMA
Fixed: 407544212 Change-Id: I47a0d650bf7eac00b4b363610922122c65cabf8a Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6425302 Commit-Queue: Ergün Erdoğmuş <ergunsh@chromium.org> Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
1 parent bac9d21 commit 9c2e34b

File tree

4 files changed

+3
-18
lines changed

4 files changed

+3
-18
lines changed

front_end/core/host/InspectorFrontendHostAPI.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -464,7 +464,6 @@ export const enum EnumeratedHistogram {
464464
// LINT.IfChange(EnumeratedHistogram)
465465
ActionTaken = 'DevTools.ActionTaken',
466466
PanelShown = 'DevTools.PanelShown',
467-
PanelShownInLocation = 'DevTools.PanelShownInLocation',
468467
SidebarPaneShown = 'DevTools.SidebarPaneShown',
469468
KeyboardShortcutFired = 'DevTools.KeyboardShortcutFired',
470469
IssueCreated = 'DevTools.IssueCreated',

front_end/core/host/UserMetrics.ts

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -51,16 +51,6 @@ export class UserMetrics {
5151
}
5252
}
5353

54-
panelShownInLocation(panelName: string, location: 'main'|'drawer'): void {
55-
const panelWithLocationName = `${panelName}-${location}`;
56-
const panelWithLocation = PanelWithLocation[panelWithLocationName as keyof typeof PanelWithLocation] || 0;
57-
InspectorFrontendHostInstance.recordEnumeratedHistogram(
58-
EnumeratedHistogram.PanelShownInLocation,
59-
panelWithLocation,
60-
PanelWithLocation.MAX_VALUE,
61-
);
62-
}
63-
6454
settingsPanelShown(settingsViewId: string): void {
6555
this.panelShown('settings-' + settingsViewId);
6656
}

front_end/devtools_compatibility.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -423,7 +423,6 @@ const EnumeratedHistogram = {
423423
LighthouseModeRun: 'DevTools.LighthouseModeRun',
424424
LighthouseCategoryUsed: 'DevTools.LighthouseCategoryUsed',
425425
PanelShown: 'DevTools.PanelShown',
426-
PanelShownInLocation: 'DevTools.PanelShownInLocation',
427426
RecordingAssertion: 'DevTools.RecordingAssertion',
428427
RecordingCodeToggled: 'DevTools.RecordingCodeToggled',
429428
RecordingCopiedToClipboard: 'DevTools.RecordingCopiedToClipboard',

front_end/ui/legacy/InspectorView.ts

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -182,11 +182,10 @@ export class InspectorView extends VBox implements ViewLocationResolver {
182182
closeDrawerButton.addEventListener(ToolbarButton.Events.CLICK, this.closeDrawer, this);
183183
this.drawerTabbedPane.addEventListener(
184184
TabbedPaneEvents.TabSelected,
185-
(event: Common.EventTarget.EventTargetEvent<EventData>) => this.tabSelected(event.data.tabId, 'drawer'), this);
185+
(event: Common.EventTarget.EventTargetEvent<EventData>) => this.tabSelected(event.data.tabId), this);
186186
const selectedDrawerTab = this.drawerTabbedPane.selectedTabId;
187187
if (this.drawerSplitWidget.showMode() !== ShowMode.ONLY_MAIN && selectedDrawerTab) {
188188
Host.userMetrics.panelShown(selectedDrawerTab, true);
189-
Host.userMetrics.panelShownInLocation(selectedDrawerTab, 'drawer');
190189
}
191190
this.drawerTabbedPane.setTabDelegate(this.tabDelegate);
192191

@@ -219,11 +218,10 @@ export class InspectorView extends VBox implements ViewLocationResolver {
219218
this.tabbedPane.registerRequiredCSS(inspectorViewTabbedPaneStyles);
220219
this.tabbedPane.addEventListener(
221220
TabbedPaneEvents.TabSelected,
222-
(event: Common.EventTarget.EventTargetEvent<EventData>) => this.tabSelected(event.data.tabId, 'main'), this);
221+
(event: Common.EventTarget.EventTargetEvent<EventData>) => this.tabSelected(event.data.tabId), this);
223222
const selectedTab = this.tabbedPane.selectedTabId;
224223
if (selectedTab) {
225224
Host.userMetrics.panelShown(selectedTab, true);
226-
Host.userMetrics.panelShownInLocation(selectedTab, 'main');
227225
}
228226
this.tabbedPane.setAccessibleName(i18nString(UIStrings.panels));
229227
this.tabbedPane.setTabDelegate(this.tabDelegate);
@@ -454,9 +452,8 @@ export class InspectorView extends VBox implements ViewLocationResolver {
454452
this.tabbedPane.headerResized();
455453
}
456454

457-
private tabSelected(tabId: string, location: 'main'|'drawer'): void {
455+
private tabSelected(tabId: string): void {
458456
Host.userMetrics.panelShown(tabId);
459-
Host.userMetrics.panelShownInLocation(tabId, location);
460457
}
461458

462459
setOwnerSplit(splitWidget: SplitWidget): void {

0 commit comments

Comments
 (0)