-
Notifications
You must be signed in to change notification settings - Fork 30
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
Chrome M121 breaks <ruby> unittest #476
Comments
@kojiishi Could you take a look at this? It seems the |
Thanks, I'll take a look. /cc @tkent-google |
So there was a spec change. The current BudouX has a logic to determine if an element is inline or block: return display === 'inline' ? DomAction.Inline : DomAction.Block; This no longer works since crbug.com/880802, as The CSS Display multi-keyword syntax also shipped at crbug.com/995106. Looking at this, |
That’s interesting! Thanks for investigating. |
A JavaScript unit test case is failing at https://github.com/google/budoux/actions/runs/7726982481/job/21064428934
Since the same test case passes on Chrome Headless M120, but not on M121, I believe the change is brought by the Chrome update.
The text was updated successfully, but these errors were encountered: