[webkit-changes] [WebKit/WebKit] 15c6ce: Make the SVG parser interpret `form feed` as white...
Ahmad Saleem
noreply at github.com
Tue Jan 23 05:35:27 PST 2024
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 15c6cea827c0761eb8c1b8638fc12568bf4170b3
https://github.com/WebKit/WebKit/commit/15c6cea827c0761eb8c1b8638fc12568bf4170b3
Author: Ahmad Saleem <ahmad.saleem792+github at gmail.com>
Date: 2024-01-23 (Tue, 23 Jan 2024)
Changed paths:
A LayoutTests/svg/transforms/svg-formFeed-as-whitespace-expected.html
A LayoutTests/svg/transforms/svg-formFeed-as-whitespace.html
A LayoutTests/svg/transforms/svg-line-tabulation-as-not-whitespace-expected.html
A LayoutTests/svg/transforms/svg-line-tabulation-as-not-whitespace.html
M Source/WebCore/svg/SVGParserUtilities.cpp
M Source/WebCore/svg/SVGParserUtilities.h
M Source/WebCore/svg/SVGStringList.cpp
Log Message:
-----------
Make the SVG parser interpret `form feed` as whitespace
https://bugs.webkit.org/show_bug.cgi?id=77755
rdar://problem/95488677
Reviewed by Anne van Kesteren.
This patch is to align WebKit with Chromium / Blink, Gecko / Firefox and Web Specification [1]:
[1] https://lists.w3.org/Archives/Public/public-svg-wg/2014AprJun/0068.html
This patch uses 'isASCIIWhite' across SVG Parser and code base to enable it to handle 'form feed'.
It is inspired by following change in Blink, which enables 'leading' and 'trailing' whitespace
in SVG Attributes [2]:
[2] https://src.chromium.org/viewvc/blink?view=revision&revision=175785
Credits to Jacob Goldstein <jacobg at adobe.com> for 'test case'.
* Source/WebCore/svg/SVGParserUtilities.cpp:
(parseGlyphName):
* Source/WebCore/svg/SVGParserUtilities.h:
(isSVGSpace): Deleted
(isSVGSpaceOrComma): Updated
(skipOptionalSVGSpaces):
(skipOptionalSVGSpacesOrDelimiter): Both templates
* Source/WebCore/svg/SVGStringList.cpp:
(SVGStringList::parse):
* LayoutTests/svg/transforms/svg-formFeed-as-whitespace.html: Add Test Case
* LayoutTests/svg/transforms/svg-formFeed-as-whitespace-expected.html: Add Test Case Expectation
* LayoutTests/svg/transforms/svg-line-tabulation-as-not-whitespace.html: Add Test Case
* LayoutTests/svg/transforms/svg-line-tabulation-as-not-whitespace-expected.html: Add Test Case Expectation
Canonical link: https://commits.webkit.org/273353@main
More information about the webkit-changes
mailing list