LibJS+LibWeb+LibWasm: Remove various `DeprecatedFlyString`s

LadybirdBrowser / ladybird

Truly independent web browser

https://ladybird.org

BSD 2-Clause "Simplified" License

22.27k stars 987 forks source link

LibJS+LibWeb+LibWasm: Remove various `DeprecatedFlyString`s #2493

Open yyny opened 21 hours ago

yyny commented 21 hours ago

Things of note:

PropertyKeys/StringOrSymbols are converted from DeprecatedFlyString to FlyString when used as function names. This is inefficient.
Various name fields have been moved to FlyString rather than relying on the implicit conversion from DeprecatedFlyString to ByteString. FlyString is probably more appropriate anyway.

ladybird-bot commented 21 hours ago

Hello!

One or more of the commit messages in this PR do not match the Ladybird code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why. Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.