Skip to content

perf(DRM): Run expiration timer only when EME is in use #8853

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 8, 2025

Conversation

tykus160
Copy link
Member

@tykus160 tykus160 commented Jul 8, 2025

There is no need to poll expiration if content is not encrypted or EME is not initialized yet.

@tykus160 tykus160 added this to the v4.16 milestone Jul 8, 2025
@tykus160 tykus160 requested review from joeyparrish, avelad and TAhub July 8, 2025 10:39
@tykus160 tykus160 added component: EME The issue involves the Encrypted Media Extensions web API type: performance A performance issue labels Jul 8, 2025
@shaka-bot
Copy link
Collaborator

Incremental code coverage: 100.00%

@avelad avelad merged commit c14e6cc into shaka-project:main Jul 8, 2025
35 of 37 checks passed
avelad pushed a commit that referenced this pull request Jul 15, 2025
There is no need to poll expiration if content is not encrypted or EME
is not initialized yet.
avelad pushed a commit that referenced this pull request Jul 15, 2025
There is no need to poll expiration if content is not encrypted or EME
is not initialized yet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: EME The issue involves the Encrypted Media Extensions web API type: performance A performance issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants