Closed Bug 626714 Opened 9 years ago Closed 9 years ago

Send startup time measurements to AMO

Categories

(Firefox for Android Graveyard :: General, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: mfinkle, Assigned: mfinkle)

References

Details

Attachments

(1 file)

Bug 623950 is using the new startup time details (see bug 522375) to send information to AMO to help improve the affects of add-ons on startup time.

Fennec should do the same, however, we need to use a different trigger. Firefox will be using the "meta data request" to send the data to AMO. Fennec doesn't use the AddonManager cache, and therefore, the request is never sent.

Fennec could just manually fire off a manual request when initializing ExtensionsView. It would be async and should hurt performance.
(In reply to comment #0)

> ExtensionsView. It would be async and should hurt performance.

... should not hurt performance
Attached patch patchSplinter Review
Simply turns on add-on cache and provides a proper URL (same as FX but uses "mobile")
Assignee: nobody → mark.finkle
Attachment #504790 - Flags: review?(dtownsend)
OS: Linux → All
Hardware: x86_64 → All
Comment on attachment 504790 [details] [diff] [review]
patch

Update the parameters in the pref but otherwise this looks good
Attachment #504790 - Flags: review?(dtownsend) → review+
pushed:
http://hg.mozilla.org/mobile-browser/rev/6fabfbf280ac
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
pushed a fixup after talking to Justin. We have to hardcode "mobile" and not use %APP% in the URL:
http://hg.mozilla.org/mobile-browser/rev/9d31f9f5a56b
Is there a way for us to verify this? If not, can somebody verify please?
I have seen the numbers
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.