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
Update GB-18030 encoding table #14716
Update GB-18030 encoding table #14716
Conversation
c33fe4f
to
c21e34d
Compare
EWS run on previous version of this PR (hash c21e34d)
|
c21e34d
to
dc7d29a
Compare
EWS run on current version of this PR (hash dc7d29a)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Temporary r+ until we can figure out a way to delete these tables.
https://bugs.webkit.org/show_bug.cgi?id=257770 rdar://110353061 Reviewed by Myles C. Maxfield. This was already done internally in ICU in rdar://107702106 This reflects changes published as GB-18030-2022 This was proposed as a change to the standard at whatwg/encoding#312 This fixes an assertion when running encoding tests on macOS Sonoma and iOS 17, and I added test coverage specific to the 18 changed code points. * LayoutTests/imported/w3c/web-platform-tests/encoding/legacy-mb-schinese/gb18030/gb18030-encoder-expected.txt: * LayoutTests/imported/w3c/web-platform-tests/encoding/legacy-mb-schinese/gb18030/gb18030-encoder.html: * Source/WTF/wtf/PlatformHave.h: * Source/WebCore/PAL/pal/text/EncodingTables.cpp: (PAL::gb18030): Canonical link: https://commits.webkit.org/264918@main
dc7d29a
to
b7d4f07
Compare
Committed 264918@main (b7d4f07): https://commits.webkit.org/264918@main Reviewed commits have been landed. Closing PR #14716 and removing active labels. |
Note: #15413 has a necessary fix after this. |
This CL imports WebKit/WebKit#14716. In case supporting GB18030-2022 causes the issues, it has been implemented behind the Gb18030_2022Enabled flag. Change-Id: Ie50d6b0c32fa18026c43414b3a8404556640b5d0 Bug: 1457267 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4710388 Reviewed-by: Koji Ishii <kojii@chromium.org> Reviewed-by: Kent Tamura <tkent@chromium.org> Commit-Queue: Yoshisato Yanagisawa <yyanagisawa@chromium.org> Cr-Commit-Position: refs/heads/main@{#1174127}
b7d4f07
dc7d29a
π wincairoπ§ͺ api-macπ π§ͺ jscπ§ͺ mac-AS-debug-wk2π tv-simπ watch