Skip to content

Change HTML Video and Audio Lazy Loading Tests from Tentative to Non-Tentative - #57754

Merged
zcorpan merged 1 commit into
web-platform-tests:masterfrom
Squarespace:media-loading-lazy-standard
Mar 23, 2026
Merged

zcorpan merged 1 commit into
web-platform-tests:masterfrom
Squarespace:media-loading-lazy-standard

Conversation

@scottjehl

Copy link
Copy Markdown
Contributor

This is intended to pair with the merge of whatwg/html#11980

  • Tentative filenames removed
  • Reflection tests are moved to their proper place and temporary HTML reflection tests deleted
@keithamus

keithamus commented Feb 12, 2026 •

Copy link
Copy Markdown
Member

I cant' tell if github's diff is being weird or if files have actually been removed here? Would you kindly double check, e.g. html/semantics/embedded-content/the-video-element/video-loading-attr-default.tentative.html

@scottjehl

Copy link
Copy Markdown
Contributor Author

That's correct. 6 HTML reflection tests are deleted as part of moving reflection tests to elements-embedded.js

The rest of it is file renaming.

@scottjehl

Copy link
Copy Markdown
Contributor Author

In the process, I noticed a couple of those video tests were mistakenly in the audio directory but they'll be removed as part of this change.

@zcorpan

zcorpan commented Mar 23, 2026

Copy link
Copy Markdown
Member

@scottjehl can you resolve conflicts?

@scottjehl

Copy link
Copy Markdown
Contributor Author

@zcorpan will do. I'll include enabling the tests in elements-embedded.js for reflection as well.

@scottjehl
scottjehl force-pushed the media-loading-lazy-standard branch from c12e386 to 12a9679 Compare March 23, 2026 17:14
@scottjehl

Copy link
Copy Markdown
Contributor Author

@zcorpan rebased on master and:

  • updated elements-embedded.js to enable the reflection tests
  • renamed tentative files that should stick around
  • deleted 4 tentative files dealing with tests covered better by the JS reflection tests (note: those are the 4 deletions, and a couple of the video tests turned out to be in the audio folder in hindsight, but are deleted here.)
@scottjehl
scottjehl requested a review from zcorpan March 23, 2026 17:23
@scottjehl

Copy link
Copy Markdown
Contributor Author

@zcorpan also, these two PRs are related. One fixes a bug in 2 tests in this PR, and another adds 2 tests.
I can make relevant updates to this PR or these depending on whether this one lands first.

@zcorpan
zcorpan merged commit a024f97 into web-platform-tests:master Mar 23, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

5 participants