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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants