[webkit-changes] [WebKit/WebKit] 1303a4: [LegacySVG] Adding 'id's of pending SVG "resource"...

[Ahmad Saleem]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1303a49bf2d3d814eca190822c033a43eab7992d
      https://github.com/WebKit/WebKit/commit/1303a49bf2d3d814eca190822c033a43eab7992d
  Author: Ahmad Saleem <ahmad.saleem792+github at gmail.com>
  Date:   2024-09-14 (Sat, 14 Sep 2024)

  Changed paths:
    A LayoutTests/svg/text/adding-id-on-pending-path-expected.html
    A LayoutTests/svg/text/adding-id-on-pending-path.html
    A LayoutTests/svg/text/removing-id-on-path-expected.html
    A LayoutTests/svg/text/removing-id-on-path.html
    M Source/WebCore/svg/SVGTextPathElement.cpp

  Log Message:
  -----------
  [LegacySVG] Adding 'id's of pending SVG "resource" references does not work for non-resources

https://bugs.webkit.org/show_bug.cgi?id=279323
rdar://problem/135509733

Reviewed by Alan Baradlay.

This patch aligns WebKit with Gecko / Firefox and Blink / Chromium.

Merge: https://chromium.googlesource.com/chromium/src.git/+/6a5978f382eca334bfab56ab7409a71e45a51bca

When add 'id' on pending resource(<path>), it does no apply to
non-resources(<textPath>). Because if need to build pending resource,
re-layout non-resources.

* Source/WebCore/svg/SVGTextPathElement.cpp:
(WebCore::SVGTextPathElement::buildPendingResource):
* LayoutTests/svg/text/adding-id-on-pending-path.html:
* LayoutTests/svg/text/adding-id-on-pending-path-expected.html:
* LayoutTests/svg/text/removing-id-on-path.html:
* LayoutTests/svg/text/removing-id-on-path-expected.html:

Canonical link: https://commits.webkit.org/283675@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications