Be explicit about LOAD_URL telemetry from Home Panels

VERIFIED FIXED in Firefox 42

Status

()

Firefox for Android
Awesomescreen
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: mfinkle, Assigned: mfinkle)

Tracking

unspecified
Firefox 44
Points:
---

Firefox Tracking Flags

(firefox42 verified, firefox43 verified, firefox44 verified)

Details

Attachments

(1 attachment)

Created attachment 8673141 [details] [diff] [review]
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.
Attachment #8673141 - Flags: review?(liuche)
Attachment #8673141 - Flags: review?(liuche) → review+

Comment 1

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/17bb5573ac93
https://hg.mozilla.org/mozilla-central/rev/17bb5573ac93
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox44: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
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
Attachment #8673141 - Flags: approval-mozilla-beta?
Attachment #8673141 - Flags: approval-mozilla-aurora?
Assignee: nobody → mark.finkle
status-firefox42: --- → affected
status-firefox43: --- → affected
Comment on attachment 8673141 [details] [diff] [review]
explicit-homepanel-loadurl v0.1

Sure, if that can help you.
Should be in 42 beta 8.
Attachment #8673141 - Flags: approval-mozilla-beta?
Attachment #8673141 - Flags: approval-mozilla-beta+
Attachment #8673141 - Flags: approval-mozilla-aurora?
Attachment #8673141 - Flags: approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/ae10da57b8db
status-firefox43: affected → fixed
https://hg.mozilla.org/releases/mozilla-beta/rev/3e94d092cb56
status-firefox42: affected → fixed

Comment 7

2 years ago
Verified as fixed on Firefox 42 Beta 8, latest Aurora and Nightly
Status: RESOLVED → VERIFIED
status-firefox42: fixed → verified
status-firefox43: fixed → verified
status-firefox44: fixed → verified
You need to log in before you can comment on or make changes to this bug.