Closed
Bug 1004889
Opened 11 years ago
Closed 11 years ago
Add UI telemetry for the main menu
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox31 verified, firefox32 verified)
VERIFIED
FIXED
Firefox 32
People
(Reporter: mfinkle, Assigned: mfinkle)
References
Details
Attachments
(1 file)
2.92 KB,
patch
|
rnewman
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
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.
Attachment #8416303 -
Flags: review?(rnewman)
Updated•11 years ago
|
Attachment #8416303 -
Flags: review?(rnewman) → review+
Updated•11 years ago
|
Assignee: nobody → mark.finkle
Status: NEW → ASSIGNED
OS: Linux → Android
Hardware: x86_64 → All
Assignee | ||
Comment 1•11 years ago
|
||
Comment 2•11 years ago
|
||
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
Assignee | ||
Comment 3•11 years ago
|
||
(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
Assignee | ||
Comment 4•11 years ago
|
||
For completeness, the Try run: https://tbpl.mozilla.org/?tree=Try&rev=41b9dae6d1a8
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 32
Assignee | ||
Comment 6•11 years ago
|
||
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
Attachment #8416303 -
Flags: approval-mozilla-aurora?
Updated•11 years ago
|
status-firefox31:
--- → affected
status-firefox32:
--- → fixed
Updated•11 years ago
|
Attachment #8416303 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 7•11 years ago
|
||
Comment 8•11 years ago
|
||
Verified as fixed in builds:
- 32.0a1 (2014-05-29);
- 31.0a2 (2014-05-29);
Device: Samsung Galaxy Nexus (Android 4.2.1).
Assignee | ||
Updated•10 years ago
|
Blocks: mobile-ui-telemetry
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•