626714 - Send startup time measurements to AMO

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

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.

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #0)

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

... should not hurt performance

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch

Simply turns on add-on cache and provides a proper URL (same as FX but uses "mobile")

Comment 3

[Dave Townsend [:mossop]]

Comment on attachment 504790 [details] [diff] [review]
patch

Update the parameters in the pref but otherwise this looks good

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

pushed:
http://hg.mozilla.org/mobile-browser/rev/6fabfbf280ac

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

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

Comment 6

[Andreea Pod]

Is there a way for us to verify this? If not, can somebody verify please?

Comment 7

[Mark Finkle (:mfinkle) (use needinfo?)]

I have seen the numbers