1214234 - Be explicit about LOAD_URL telemetry from Home Panels

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Awesomescreen, defect)

Description

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

Attachment: explicit-homepanel-loadurl v0.1

We have been using the Session telemetry to figure out the LOAD_URL/LIST_ITEM source panel. This should work fine, but I am a little worried that my post-processing scripts could be getting confused by the fact that we can display Home Panels on top of Home Panels, creating a nesting of panel sessions.

I mean, it's OK to open about:home (which creates a Session) and then tap the URL bar (which opens a different Home Panel and creates a different Session).

By tagging the Telemetry with an Extra, we can be more accurate.

Comment 1

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/17bb5573ac93

Comment 2

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/17bb5573ac93

Comment 3

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

Comment on attachment 8673141 [details] [diff] [review]
explicit-homepanel-loadurl v0.1

Approval Request Comment
[Feature/regressing bug #]:
[User impact if declined]: We get more accurate LOAD_URL telemetry and be more sure about which parts fo the app are most active.
[Describe test coverage new/current, TreeHerder]: On Nightly
[Risks and why]: Low, just added an Extra hint
[String/UUID change made/needed]: none

Comment 4

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8673141 [details] [diff] [review]
explicit-homepanel-loadurl v0.1

Sure, if that can help you.
Should be in 42 beta 8.

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/ae10da57b8db

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-beta/rev/3e94d092cb56

Comment 7

[u421692]

Verified as fixed on Firefox 42 Beta 8, latest Aurora and Nightly