1008386 - Add thumbnail position to griditem loadurl probe

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Awesomescreen, defect)

Description

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

Attachment: uitelemetry-grid-position v0.1

We can add the position of the thumbnail that was tapped to open a URL. It might be useful for detecting usage patterns. It's pretty easy to do as well.

The "tablet" flag (bug 1007836) can be used to help us switch analyses between phone and tablet layouts.

Comment 1

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

https://hg.mozilla.org/integration/fx-team/rev/347544269c33

Comment 2

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/347544269c33

Comment 3

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

Comment on attachment 8420306 [details] [diff] [review]
uitelemetry-grid-position v0.1

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: We want to do before and after testing with telemetry
Testing completed (on m-c, etc.): Works on m-c
Risk to taking this patch (and alternatives if risky): low, but will likely need a rebased patch
String or IDL/UUID changes made by this patch: none

Comment 4

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

https://hg.mozilla.org/releases/mozilla-aurora/rev/47540d80872a

Comment 5

[Flaviu Cos]

SendUIEvent: action = loadurl.1 method = griditem timestamp = 18599525 extras = 0
SendUIEvent: action = loadurl.1 method = griditem timestamp = 18599525 extras = 1
SendUIEvent: action = loadurl.1 method = griditem timestamp = 18599525 extras = 2
SendUIEvent: action = loadurl.1 method = griditem timestamp = 18599525 extras = 3
SendUIEvent: action = loadurl.1 method = griditem timestamp = 18599525 extras = 4
SendUIEvent: action = loadurl.1 method = griditem timestamp = 18599525 extras = 5

Verified as fixed in builds:
- 32.0a1 (2014-05-30);
- 31.0a2 (2014-05-30);

Device: Alcatel One Touch 8008D (Android 4.1.2)