Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Remove Event.wrapper for MediaNotification telemetry #24603

Closed
Alexandru2909 opened this issue Apr 5, 2022 · 1 comment
Closed

Remove Event.wrapper for MediaNotification telemetry #24603

Alexandru2909 opened this issue Apr 5, 2022 · 1 comment
Assignees

Comments

@Alexandru2909
Copy link
Contributor

Alexandru2909 commented Apr 5, 2022

Breakout from #21292.

┆Issue is synchronized with this Jira Task

@Alexandru2909 Alexandru2909 self-assigned this Apr 5, 2022
@github-actions github-actions bot added the needs:triage Issue needs triage label Apr 5, 2022
@Alexandru2909 Alexandru2909 removed the needs:triage Issue needs triage label Apr 5, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Apr 5, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Apr 6, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Apr 7, 2022
@Alexandru2909 Alexandru2909 added the eng:qa:needed QA Needed label Apr 8, 2022
@LaurentiuApahideanSV
Copy link

I tested the issue on Firefox Nightly 2022-4-12 by interacting with videos and observing the events.
When a video or audio starts and is paused
{
"timestamp": 15714,
"category": "media_state",
"name": "play"
},

{
"timestamp": 22335,
"category": "media_state",
"name": "pause"
},

When a video is put in fullscreen mode
{
"timestamp": 26244,
"category": "media_state",
"name": "fullscreen"
},
When a video is put in picture in picture mode
{
"timestamp": 28956,
"category": "media_state",
"name": "picture_in_picture"
},
When audio is started and paused from the push notification
{
"timestamp": 14138,
"category": "media_notification",
"name": "pause"
},

{
"timestamp": 15384,
"category": "media_notification",
"name": "play"
},

When audio is started and paused from the tabs tray
{
"timestamp": 7515,
"category": "tab",
"name": "media_play"
},

{
"timestamp": 8910,
"category": "tab",
"name": "media_pause"
},
Glean Dashboard

Device used:

  • OnePlus 6T (Android 9)

@LaurentiuApahideanSV LaurentiuApahideanSV added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Apr 12, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Apr 19, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Apr 19, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Apr 19, 2022
Alexandru2909 pushed a commit to Alexandru2909/fenix that referenced this issue Apr 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants