1302025 - (EME_Telemetry) Add telemetry to know how many users use EME on Fennec for DRM

Status: RESOLVED DUPLICATE of bug 1304207

(Firefox for Android Graveyard :: Audio/Video, defect)

Description

[James Cheng[:JamesCheng]]

Once we phased in the EME API support, we should add some telemetry to know how many users may use these API for watching the DRM content on Fennec.

Comment 1

[James Cheng[:JamesCheng]]

I found ```Bug 1304207 - [EME] Add telemetry tracking CDM usage``` is doing the same thing but I want to separate the WideVine into two cases for Desktop or Fennec.

I will do this based on bug 1304207.

Comment 2

[James Cheng[:JamesCheng]]

Attachment: Bug 1302025 - (EME_Telemetry) Add telemetry to know how many users use EME on Fennec for DRM

Review commit: https://reviewboard.mozilla.org/r/101136/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/101136/

Comment 3

[James Cheng[:JamesCheng]]

Hi gerald, 

Please help to review this patch, 

the purpose is to record the use of widevine drm on android platform.

I don't know if I need data review guys to review this patch also... what do you think?

Thank you!

Comment 4

[Gerald Squelart (he/him) (not at Mozilla since 2022-09-15)]

Comment on attachment 8822132 [details]
Bug 1302025 - (EME_Telemetry) Add telemetry to know how many users use EME on Fennec for DRM

https://reviewboard.mozilla.org/r/101136/#review101644

Code looks good, I was about to give r+... But then I remembered: Telemetry is already capturing this distinction!

Go to the telemetry page and you can choose "Desktop" and/or "Mobile".
( https://telemetry.mozilla.org/new-pipeline/dist.html#measure=VIDEO_CDM_CREATED&product=Fennec&sanitize=0 )
So I think this bug might be unnecessary.

Now, looking at the "Mobile" numbers, it seems nothing related to "CDM" happens on Mobile!? (Zero data)
I'd suggest you investigate that first, see if you can actually get to that path on Fennec.

Comment 5

[James Cheng[:JamesCheng]]

Thank you gerald for your reminding.

Mobile Widevine has not beend enabled yet so the zero data is normal.

Mobile Widevine CDM will be enabled by bug 1325332.