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.

Sign up for GitHub

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

Jump to bottom

A reduced test case for a WebAnimation WPT crashes #2423

Open

shlyakpavel opened this issue Nov 18, 2024 · 0 comments · May be fixed by #2424

Contributor

shlyakpavel commented Nov 18, 2024

I've noticed we're crashing on accumulation-per-property-001.html WPT
The test is huge, and involves thousands of lines of code.
This is why I made a reduced test case that crashes Ladybird. If we fix it, we're likely to significantly improve our WPT score as this crash involves 310 subtests at accumulation-per-property-001.html, 310 subtests at addition-per-property-001.html and 385 subtests at interpolation-per-property-001.html

const element = document.createElement('div');
element.style.fontLanguageOverride = '"eng"';
document.body.appendChild(element);

const animation = element.animate(
  [
    { fontLanguageOverride: '"eng"' },
    { fontLanguageOverride: '"xyz"' }
  ],
  { duration: 1000, iterations: 1 }
);
animation.play();

tcl3 linked a pull request

that will close this issue

LibWeb: Use correct specifier to pad font language override value #2424

Open

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