corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1593953 - Make Touch Bar telemetry optional in execCommand. r=mikedeboer

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D51790
This commit is contained in:
Harry Twyford
2019-11-05 12:54:19 +00:00
parent 10f0dacdb9
commit 9f9609e5fa
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
browser/components/touchbar/MacTouchBar.js
View File

@@ -35,7 +35,7 @@ XPCOMUtils.defineLazyServiceGetter(
* TouchBarInput. * TouchBarInput.
* @param {string} commandName * @param {string} commandName
* A XUL command. * A XUL command.
* @param {string} telemetryKey * @param {string} [telemetryKey]
* A string describing the command, sent for telemetry purposes. * A string describing the command, sent for telemetry purposes.
* Intended to be shorter and more readable than the XUL command. * Intended to be shorter and more readable than the XUL command.
*/ */
@@ -47,6 +47,11 @@ function execCommand(commandName, telemetryKey) {
if (command) { if (command) {
command.doCommand(); command.doCommand();
} }
if (!telemetryKey) {
return;
}
let telemetry = Services.telemetry.getHistogramById( let telemetry = Services.telemetry.getHistogramById(
"TOUCHBAR_BUTTON_PRESSES" "TOUCHBAR_BUTTON_PRESSES"
); );