Skip to content

fix: ensure that requestMediaKeySystemAccess resolves - #34891

Merged
VerteDinde merged 1 commit into
20-x-yfrom
trop/20-x-y-bp-fix-ensure-that-requestmediakeysystemaccess-resolves-1657612172164
Jul 12, 2022
Merged

fix: ensure that requestMediaKeySystemAccess resolves#34891
VerteDinde merged 1 commit into
20-x-yfrom
trop/20-x-y-bp-fix-ensure-that-requestmediakeysystemaccess-resolves-1657612172164

Conversation

@trop

@trop trop Bot commented Jul 12, 2022

Copy link
Copy Markdown
Contributor

Backport of #34886

See that PR for details.

Notes: Ensures that requestMediaKeySystemAccess eventually resolves / rejects

When widevine was disabled at the build level we never dealt with the callback passed into GetSupportedKeySystems.  This was ok until requests became marked pending in https://chromium-review.googlesource.com/c/chromium/src/+/3430502 until the callback was called.  This resulted in a promise never resolving / rejecting and certain media websites (E.g. spotify) hanging on load waiting for a signal that would never arrive.
@electron-cation electron-cation Bot added the new-pr 🌱 PR opened recently label Jul 12, 2022
@trop
trop Bot requested a review from MarshallOfSound July 12, 2022 07:49
@trop trop Bot added 20-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Jul 12, 2022
@electron-cation electron-cation Bot removed the new-pr 🌱 PR opened recently label Jul 12, 2022
@VerteDinde
VerteDinde merged commit 603a4af into 20-x-y Jul 12, 2022
@VerteDinde
VerteDinde deleted the trop/20-x-y-bp-fix-ensure-that-requestmediakeysystemaccess-resolves-1657612172164 branch July 12, 2022 17:56
@release-clerk

release-clerk Bot commented Jul 12, 2022

Copy link
Copy Markdown

Release Notes Persisted

Ensures that requestMediaKeySystemAccess eventually resolves / rejects

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

20-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants