Closed Bug 1378867 Opened 4 years ago Closed 3 years ago
Use cssparser's identifier and string serialization routines in Stylo error reporter
Per https://bugzilla.mozilla.org/show_bug.cgi?id=1352669#c13: The routines in error_reporter.rs were ported from the C++ versions in nsStyleUtil.cpp. They are very similar to the serialize_identifier and CssStringWriter APIs that exist in rust-cssparser, except that they use numerical escapes for a wider set of characters. This is tested in test_parser_diagnostics_unprintables.html, but it would be worth figuring out if we could remove the duplicate code and settle on one implementation in the future.
This was done in https://github.com/servo/servo/commit/ad4fe4318794c51ea255ef486baab08f04f4db3e#diff-6d5f63742420613722936477261cb39e.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.