Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 15c6cea827c0761eb8c1b8638fc12568bf4170b3
      
https://github.com/WebKit/WebKit/commit/15c6cea827c0761eb8c1b8638fc12568bf4170b3
  Author: Ahmad Saleem <[email protected]>
  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  <[email protected]> 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


_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to