Closed Bug 1864027 Opened 11 months ago Closed 11 months ago

[wmfme] use `IMFExtendedDRMTypeSupport::IsTypeSupportedEx` to check the capabilites for PlayReady

Categories

(Core :: Audio/Video: Playback, task, P1)

task

Tracking

()

RESOLVED FIXED
121 Branch
Tracking Status
firefox121 --- fixed

People

(Reporter: alwu, Assigned: alwu)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

We found that PlayReady doesn't return correct capabilies when we call IMFContentDecryptionModuleFactory::IsTypeSupported.

After confirming with Microsoft, they told me that for PlayReady we need to use another method IMFExtendedDRMTypeSupport::IsTypeSupportedEx. But Widevine implements IMFContentDecryptionModuleFactory::IsTypeSupported well.

We found that PlayReady doesn't return correct capabilies when we call
IMFContentDecryptionModuleFactory::IsTypeSupported.

After confirming with Microsoft, they told us that for PlayReady we need
to use another method IMFExtendedDRMTypeSupport::IsTypeSupportedEx.

In contrast, Widevine implements IsTypeSupported well, so it doesn't
need to use another method.

That workaround was added for PlayReady before because it didn't return
correct result from IsTypeSupport. As we don't use IsTypeSupport for
PlayReady anymore and Wideinve seems always returning correct result, we
can remove that workaround.

Depends on D193231

Attachment #9362892 - Attachment description: Bug 1864027 - part1 : use `IMFExtendedDRMTypeSupport::IsTypeSupportedEx` to check the capabilites for PlayReady. → Bug 1864027 - part1 : use `IMFExtendedDRMTypeSupport::IsTypeSupportedEx` to check the capabilites for PlayReady.
Pushed by alwu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/5d297cad78f4 part1 : use `IMFExtendedDRMTypeSupport::IsTypeSupportedEx` to check the capabilites for PlayReady. r=jolin https://hg.mozilla.org/integration/autoland/rev/f0567d6897d8 part2 : remove workaround for checking HWDRM. r=jolin
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
Regressions: 1851914
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: