Bug 1063561 - Convert Addon Manager telemetry timings to use Components.utils.now(); r=irvingreid

This commit is contained in:
Manu Jain 2014-09-19 14:04:49 -04:00
parent 094e8228cc
commit 66dc378c73
2 changed files with 6 additions and 6 deletions

View File

@ -2431,9 +2431,9 @@ this.AddonManagerPrivate = {
// Start a timer, record a simple measure of the time interval when
// timer.done() is called
simpleTimer: function(aName) {
let startTime = Date.now();
let startTime = Cu.now();
return {
done: () => this.recordSimpleMeasure(aName, Date.now() - startTime)
done: () => this.recordSimpleMeasure(aName, Math.round(Cu.now() - startTime))
};
},

View File

@ -1494,7 +1494,7 @@ XPIState.prototype = {
*/
getModTime(aFile, aId) {
let changed = false;
let scanStarted = Date.now();
let scanStarted = Cu.now();
// For an unknown or enabled add-on, we do a full recursive scan.
if (!('scanTime' in this) || this.enabled) {
logger.debug('getModTime: Recursive scan of ' + aId);
@ -1539,7 +1539,7 @@ XPIState.prototype = {
}
}
// Record duration of file-modified check
XPIProvider.setTelemetry(aId, "scan_MS", Date.now() - scanStarted);
XPIProvider.setTelemetry(aId, "scan_MS", Math.round(Cu.now() - scanStarted));
return changed;
},
@ -3539,10 +3539,10 @@ this.XPIProvider = {
}
// Telemetry probe added around getInstallState() to check perf
let telemetryCaptureTime = Date.now();
let telemetryCaptureTime = Cu.now();
let installChanged = XPIStates.getInstallState();
let telemetry = Services.telemetry;
telemetry.getHistogramById("CHECK_ADDONS_MODIFIED_MS").add(Date.now() - telemetryCaptureTime);
telemetry.getHistogramById("CHECK_ADDONS_MODIFIED_MS").add(Math.round(Cu.now() - telemetryCaptureTime));
if (installChanged) {
updateReasons.push("directoryState");
}