Closed Bug 713119 Opened 8 years ago Closed 8 years ago

Make AddonRepository.getAddonsByIDs() not send performance data

Categories

(SeaMonkey :: Preferences, defect)

defect
Not set

Tracking

(seamonkey2.8 fixed)

RESOLVED FIXED
seamonkey2.9
Tracking Status
seamonkey2.8 --- fixed

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

References

Details

Attachments

(1 file)

+++ 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.

Original changeset: http://hg.mozilla.org/mozilla-central/rev/259bae11858b

From the Error Console of a build without this patch:

Timestamp: 22.12.2011 23:29:58
Error: formatURL: Couldn't find value for key: TIME_MAIN
Source File: file:///e:/mozilla-src/seamonkey-central/mozilla/dist/bin/components/nsURLFormatter.js
Line: 131

Timestamp: 22.12.2011 23:29:58
Error: formatURL: Couldn't find value for key: TIME_FIRST_PAINT
Source File: file:///e:/mozilla-src/seamonkey-central/mozilla/dist/bin/components/nsURLFormatter.js
Line: 131

Timestamp: 22.12.2011 23:29:58
Error: formatURL: Couldn't find value for key: TIME_SESSION_RESTORED
Source File: file:///e:/mozilla-src/seamonkey-central/mozilla/dist/bin/components/nsURLFormatter.js
Line: 131
Attachment #583945 - Flags: review?(bugspam.Callek)
Attachment #583945 - Flags: approval-comm-aurora?
Attachment #583945 - Flags: review?(bugspam.Callek)
Attachment #583945 - Flags: review+
Attachment #583945 - Flags: approval-comm-aurora?
Attachment #583945 - Flags: approval-comm-aurora+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.9
You need to log in before you can comment on or make changes to this bug.