1828897 - Record why GMP installation fails

Status: RESOLVED FIXED

(Core :: Audio/Video: GMP, task, P3)

Description

[Andrew Osmond [:aosmond] (he/him)]

We don't record why the installation fails, only why the download fails. We also don't track the Widevine plugin, which is the plugin we are seeing users fail to upgrade sometimes, even on nightly.

Comment 1

[Andrew Osmond [:aosmond] (he/him)]

Attachment: Bug 1828897 - Include Widevine prefs and installation failure reason in crash reports.

We currently include most information for OpenH264 installs, but not for
Widevine. We also don't include the reason why the install fails. There
are installations crashing with an old version of Widevine and we would
like to understand why they never updated.

Comment 2

[Andrew Osmond [:aosmond] (he/him)]

Attachment: Data collection request.md

Comment 3

[Chris H-C :chutten]

Comment on attachment 9329266 [details]
Data collection request.md

DATA COLLECTION REVIEW RESPONSE:

Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?

Yes.

Is there a control mechanism that allows the user to turn the data collection on and off?

Yes. This collection can be controlled through Firefox's Preferences.

If the request is for permanent data collection, is there someone who will monitor the data over time?

Yes, Andrew Osmond is responsible.

Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 2, Interaction.

Is the data collection request for default-on or default-off?

Default on for all channels.

Does the instrumentation include the addition of any new identifiers?

No.

Is the data collection covered by the existing Firefox privacy notice?

Yes.

Does the data collection use a third-party collection tool?

No.

---

Result: datareview+

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cc6c3e673cbe
Include Widevine prefs and installation failure reason in crash reports. r=media-playback-reviewers,alwu

Comment 5

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/cc6c3e673cbe

Comment 6

[Andrew Osmond [:aosmond] (he/him)]

Comment on attachment 9329265 [details]
Bug 1828897 - Include Widevine prefs and installation failure reason in crash reports.

Beta/Release Uplift Approval Request

• User impact if declined: May take longer to diagnose Widevine update failures
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Just exposes a few prefs via telemetry/crash reports
• String changes made/needed:
• Is Android affected?: No

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9329265 [details]
Bug 1828897 - Include Widevine prefs and installation failure reason in crash reports.

Approved for 113.0b8.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/9ff5e93cff6e