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

Bug 1633456 - Remove the TelemetryStorage.addPendingPing method. r=chutten

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D199602
This commit is contained in:
jingz
2024-01-25 20:24:24 +00:00
parent d13fa84db8
commit 44b1a13a8d
2 changed files with 1 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
toolkit/components/telemetry/app/TelemetryControllerParent.sys.mjs
View File

@@ -666,7 +666,7 @@ var Impl = {
// Previous aborted-session might have been with a canary client ID.
// Don't send it.
if (ping.clientId != Utils.knownClientID) {
await lazy.TelemetryStorage.addPendingPing(ping);
await lazy.TelemetryStorage.savePendingPing(ping);
await lazy.TelemetryArchive.promiseArchivePing(ping);
}
} catch (e) {

23
toolkit/components/telemetry/app/TelemetryStorage.sys.mjs
View File

@@ -405,17 +405,6 @@ export var TelemetryStorage = {
return TelemetryStorageImpl.savePing(ping, overwrite);
},
/**
* Add a ping to the saved pings directory so that it gets saved
* and sent along with other pings.
*
* @param {Object} pingData The ping object.
* @return {Promise} A promise resolved when the ping is saved to the pings directory.
*/
addPendingPing(pingData) {
return TelemetryStorageImpl.addPendingPing(pingData);
},
/**
* Remove the file for a ping
*
@@ -1470,18 +1459,6 @@ var TelemetryStorageImpl = {
return file;
},
/**
* Add a ping to the saved pings directory so that it gets saved
* and sent along with other pings.
* Note: that the original ping file will not be modified.
*
* @param {Object} ping The ping object.
* @return {Promise} A promise resolved when the ping is saved to the pings directory.
*/
addPendingPing(ping) {
return this.savePendingPing(ping);
},
/**
* Remove the file for a ping
*