Skip to content

Conversation

@danielhjacobs
Copy link
Contributor

@danielhjacobs danielhjacobs commented Dec 11, 2025

This can be used for tests that check if polyfilling occur too and for those should eliminate the need to wait for ruffle-embed/ruffle-object to exist before continuing but at the same time for those tests perhaps checking those elements exist may be considered a better condition for which to wait anyway, since that's when the tests should have their expected.html anyway.

@danielhjacobs danielhjacobs added A-web Area: Web & Extensions T-chore Type: Chore (like updating a dependency, it's gotta be done) labels Dec 11, 2025
@danielhjacobs danielhjacobs marked this pull request as draft December 11, 2025 20:37
@danielhjacobs danielhjacobs force-pushed the proof-of-concept-polyfill-finished branch 4 times, most recently from 2d0e22c to 7e75a1d Compare December 12, 2025 15:05
@danielhjacobs danielhjacobs force-pushed the proof-of-concept-polyfill-finished branch from 7e75a1d to 7c01059 Compare December 12, 2025 15:12
@danielhjacobs danielhjacobs changed the title web: Add way to check polyfill finished for tests and use for one test web: Add way to check polyfill finished for tests that check for no polyfilling Dec 12, 2025
@danielhjacobs danielhjacobs marked this pull request as ready for review December 12, 2025 15:15
@danielhjacobs danielhjacobs changed the title web: Add way to check polyfill finished for tests that check for no polyfilling web: Add way to check for initialization for tests that check for no polyfilling Dec 12, 2025
@danielhjacobs danielhjacobs added T-refactor Type: Refactor / Cleanup and removed T-chore Type: Chore (like updating a dependency, it's gotta be done) labels Dec 12, 2025
@danielhjacobs
Copy link
Contributor Author

danielhjacobs commented Dec 12, 2025

Note: This adds to the PublicAPI a new boolean initialized, which is set to true when Ruffle has done all its polyfilling. That's arguably a good thing, I can see this variable being useful to certain people.

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

Labels

A-web Area: Web & Extensions T-refactor Type: Refactor / Cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant