706797 - Make AddonRepository.getAddonsByIDs() not send performance data

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Mark Banner (:standard8)]

+++ This bug was initially created as a clone of Bug #682356 +++

Currently, the AddonRepository APIs for searching (specifically getAddonsByIDs()) automatically send performance metrics to the server.

Per discussion in bug 534956 (implementing add-on sync), having Sync query these APIs will have the undesired effect of skewing the metrics data.

AddonRepository.getAddonsByIDs() should be modified so that it optionally doesn't send performance data as part of the request and Sync can call it without skewing metrics data.

The attached patch adds an argument to getAddonsByIDs() to control sending of performance data. Patch notes:

* I purposefully made the API backwards compatible. Typically callbacks are passed last. I didn't want to go down the refactor hole.
* Note that the lack of an argument defaults to a true value.
* Is the '=== undefined' a suitable replacement for checking the length of the arguments pseudo-array?
* I still need to write the additional tests.

Comment 1

[Mark Banner (:standard8)]

Attachment: The fix

Ports the change from bug 682356

Comment 2

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 578266 [details] [diff] [review]
The fix

Looks good to me - thanks,

-Mike

Comment 3

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/2ffa45bc67d2