Closed Bug 741042 Opened 8 years ago Closed 8 years ago

Add telemetry support for timing XUL profile import

Categories

(Firefox for Android :: General, defect)

x86
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 14
Tracking Status
blocking-fennec1.0 --- beta+

People

(Reporter: mfinkle, Assigned: gcp)

References

Details

Attachments

(1 file)

We'd like to get an idea of how time consuming the XUL profile import can be on various devices and profile sizes.
We do not have Telemetry support in Java, do we?
Depends on: 741281
Assignee: nobody → gpascutto
This collects the statistics from the XUL profile. It adds one extra query to get the amount of history entries, but the overhead of that seems to be negligible.

The question is if we will ever get any of this Telemetry, given that Telemetry doesn't send on every startup for every user, *and* we only collect this once. From what I understood from Taras, Telemetry is persistent now and requires the following to work:

a) shutdown firefox cleanly
b) have the user opt in
c) have telemetry turn on subsequent start and not have persistent data from before overwritten

I'm not sure what (c) means exactly, but I think that as we never overwrite these values (except potentially history count, but then its always with the same value!), they will be kept in sessionHistograms.dat.4little and will get sent if Telemetry ever decides to send its stuff.

Do we save and persist Telemetry if the user hasn't opted in yet?
Attachment #614547 - Flags: review?(blassey.bugs)
Attachment #614547 - Flags: review?(blassey.bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/89e69819fa98
https://hg.mozilla.org/mozilla-central/rev/d5bc02349369
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
You need to log in before you can comment on or make changes to this bug.