Bug 1566891 - Remove expired telemetry probes for export options. r=pbro

Differential Revision: https://phabricator.services.mozilla.com/D55073
This commit is contained in:
Razvan Caliman
2019-12-03 11:08:55 +00:00
parent f532e08a47
commit 386e3c12eb
4 changed files with 1 additions and 109 deletions

View File

@@ -23,14 +23,6 @@ loader.lazyRequireGetter(
const ChangesApp = createFactory(require("./components/ChangesApp"));
const { getChangesStylesheet } = require("./selectors/changes");
const {
TELEMETRY_SCALAR_CONTEXTMENU_COPY_DECLARATION,
TELEMETRY_SCALAR_CONTEXTMENU_COPY_RULE,
TELEMETRY_SCALAR_COPY_ALL_CHANGES,
TELEMETRY_SCALAR_COPY_RULE,
} = require("./constants");
const { resetChanges, trackChange } = require("./actions/changes");
class ChangesView {
@@ -153,7 +145,6 @@ class ChangesView {
*/
copyAllChanges() {
this.copyChanges();
this.telemetry.scalarAdd(TELEMETRY_SCALAR_COPY_ALL_CHANGES, 1);
}
/**
@@ -203,7 +194,6 @@ class ChangesView {
const isRemoved = element.classList.contains("diff-remove");
const text = isRemoved ? `/* ${name}: ${value}; */` : `${name}: ${value};`;
clipboardHelper.copyString(text);
this.telemetry.scalarAdd(TELEMETRY_SCALAR_CONTEXTMENU_COPY_DECLARATION, 1);
}
/**
@@ -213,11 +203,8 @@ class ChangesView {
*
* @param {String} ruleId
* Rule id of the target CSS rule.
* @param {Boolean} usingContextMenu
* True if the handler is invoked from the context menu.
* (Default) False if invoked from the button.
*/
async copyRule(ruleId, usingContextMenu = false) {
async copyRule(ruleId) {
const inspectorFronts = await this.inspector.getAllInspectorFronts();
for (const inspectorFront of inspectorFronts) {
@@ -229,12 +216,6 @@ class ChangesView {
break;
}
}
if (usingContextMenu) {
this.telemetry.scalarAdd(TELEMETRY_SCALAR_CONTEXTMENU_COPY_RULE, 1);
} else {
this.telemetry.scalarAdd(TELEMETRY_SCALAR_COPY_RULE, 1);
}
}
/**