[webkit-changes] [WebKit/WebKit] e1e511: Reduce calls to AffineTransform::multiply()

[Simon Fraser]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: e1e511077e89c748d975eae1d1637e110fa805d8
      https://github.com/WebKit/WebKit/commit/e1e511077e89c748d975eae1d1637e110fa805d8
  Author: Simon Fraser <simon.fraser at apple.com>
  Date:   2024-12-13 (Fri, 13 Dec 2024)

  Changed paths:
    M Source/WebCore/platform/graphics/transforms/AffineTransform.cpp
    M Source/WebCore/platform/graphics/transforms/AffineTransform.h
    M Source/WebCore/svg/SVGGraphicsElement.cpp

  Log Message:
  -----------
  Reduce calls to AffineTransform::multiply()
https://bugs.webkit.org/show_bug.cgi?id=284596
rdar://141404759

Reviewed by Abrar Rahman Protyasha.

In SVGGraphicsElement::animatedLocalTransform() we can avoid the `matrix *= transform().concatenate()`
if the transform list is empty.

We can also avoid the multiply in `AffineTransform::makeRotation()` if the center is zero.

* Source/WebCore/platform/graphics/transforms/AffineTransform.cpp:
(WebCore::AffineTransform::makeRotation):
* Source/WebCore/platform/graphics/transforms/AffineTransform.h:
(WebCore::AffineTransform::makeRotation): Deleted.
* Source/WebCore/svg/SVGGraphicsElement.cpp:
(WebCore::SVGGraphicsElement::animatedLocalTransform const):

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



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