Closed Bug 1320730 Opened 4 years ago Closed 4 years ago

Send different panel ID for activity stream panel

Categories

(Firefox for Android Graveyard :: General, defect, P1)

All
Android
defect

Tracking

(firefox54 fixed)

RESOLVED FIXED
Firefox 54
Tracking Status
firefox54 --- fixed

People

(Reporter: sebastian, Assigned: ahunt)

References

Details

(Whiteboard: [MobileAS])

Attachments

(2 files)

Follow-up from bug 1301468.

We should send a different panel ID (not top sites) when showing the activity stream panel:
https://dxr.mozilla.org/mozilla-central/source/mobile/android/base/java/org/mozilla/gecko/home/HomePager.java?q=path%3AHomePager&redirect_type=single#545

Currently we manage our own session for activity stream. This might be able to replace the session; however start/stop behavior is different - so this needs some testing.
Assignee: nobody → ahunt
Iteration: --- → 1.15
Priority: P2 → P1
The initial patch is quite hacky - I'm still figuring out how to test this properly, but AFAICT the session is ended as soon as the HomePager is hidden (via HomePager.unload(), called from BrowserApp.hideHomePager(), which should cover our restart-to-enable-AS scenario).
We don't actually start a session on startup, or even when opening about:home. The first time a session is "started" is when you swipe left/right. I'll definitely try to fix that as part of this bug.

As far as I can tell we wouldn't need to send any new data, we could detect AS panel usage with:
WHERE sessions like '%homepanel.1:4becc86b-41eb-429a-a042-88fe8b5a094e%'
AND experiments like '%activity-stream%'

However there seems to be an issue with experiments not being transmitted and/or stored in our telemetry DB, so the above doesn't work (i.e. experiments is empty for all telemetry entries), so we probably will need to send a different session ID here.
Comment on attachment 8834606 [details]
Bug 1320730 - Ensure homepanel telemetry session is set when panel initially loaded

https://reviewboard.mozilla.org/r/110456/#review111918
Attachment #8834606 - Flags: review?(s.kaspari) → review+
Comment on attachment 8834191 [details]
Bug 1320730 - Send different panelID for topsites when AS is enabled

https://reviewboard.mozilla.org/r/110222/#review111920
Attachment #8834191 - Flags: review?(s.kaspari) → review+
Pushed by ahunt@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/834561bae53c
Ensure homepanel telemetry session is set when panel initially loaded r=sebastian
https://hg.mozilla.org/integration/autoland/rev/0693364c42cc
Send different panelID for topsites when AS is enabled r=sebastian
https://hg.mozilla.org/mozilla-central/rev/834561bae53c
https://hg.mozilla.org/mozilla-central/rev/0693364c42cc
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.