1004889 - Add UI telemetry for the main menu

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

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

Attachment: uitelemetry-menu v0.1

Simple patch that logs an event for any Main menu action. The Android resource name for the action is sent as an Extra.

This type of tracking will be useful for learning about what menu actions are used more than others.

I don't think these events will be useful for deep learning on a specific type of action, but we can use specific probes for those situations.

Comment 1

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

https://hg.mozilla.org/integration/fx-team/rev/0610c2dcc6f5

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Backed out because Android Armv6 robocop-1 went perma-fail starting with this push.
https://hg.mozilla.org/integration/fx-team/rev/fb88b8685827

https://tbpl.mozilla.org/php/getParsedLog.php?id=38947512&tree=Fx-Team

Comment 3

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

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #2)
> Backed out because Android Armv6 robocop-1 went perma-fail starting with
> this push.
> https://hg.mozilla.org/integration/fx-team/rev/fb88b8685827
> 
> https://tbpl.mozilla.org/php/getParsedLog.php?id=38947512&tree=Fx-Team

Try run was green. Retriggers on the two failing rc1 were both green and two pushes that ran before the backout were also green.

Relanded
https://hg.mozilla.org/integration/fx-team/rev/cfd4a86b9ea8

Comment 4

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

For completeness, the Try run: https://tbpl.mozilla.org/?tree=Try&rev=41b9dae6d1a8

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/cfd4a86b9ea8

Comment 6

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

Comment on attachment 8416303 [details] [diff] [review]
uitelemetry-menu v0.1

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: We miss out on some "before changes" telemetry that could show affect of Fx32 changes
Testing completed (on m-c, etc.): Working fine on m-c
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none

Comment 7

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

https://hg.mozilla.org/releases/mozilla-aurora/rev/306496a231fe

Comment 8

[Flaviu Cos]

Verified as fixed in builds:
- 32.0a1 (2014-05-29);
- 31.0a2 (2014-05-29);
Device: Samsung Galaxy Nexus (Android 4.2.1).