Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7b821671deb901620c446fff8f52fa6b11973691
      
https://github.com/WebKit/WebKit/commit/7b821671deb901620c446fff8f52fa6b11973691
  Author: Tyler Wilcock <tyle...@apple.com>
  Date:   2025-04-29 (Tue, 29 Apr 2025)

  Changed paths:
    M Source/WebCore/accessibility/AXCoreObject.h
    M Source/WebCore/accessibility/AccessibilityAttachment.cpp
    M Source/WebCore/accessibility/AccessibilityImageMapLink.cpp
    M Source/WebCore/accessibility/AccessibilityNodeObject.cpp
    M Source/WebCore/accessibility/AccessibilitySVGObject.cpp
    M Source/WebCore/accessibility/AccessibilityTable.cpp

  Log Message:
  -----------
  AX: Many places where instances of AccessibilityText are created 
unnecessarily copy a string where it could be moved instead
https://bugs.webkit.org/show_bug.cgi?id=292297
rdar://150315733

Reviewed by Chris Fleizach.

Also pass AccessibilityTextSource by value instead of by reference. The type of 
this enum is an int, and it's actually
less efficient to pass small types by-reference.

* Source/WebCore/accessibility/AXCoreObject.h:
(WebCore::AccessibilityText::AccessibilityText):
* Source/WebCore/accessibility/AccessibilityAttachment.cpp:
(WebCore::AccessibilityAttachment::accessibilityText const):
* Source/WebCore/accessibility/AccessibilityImageMapLink.cpp:
(WebCore::AccessibilityImageMapLink::accessibilityText const):
* Source/WebCore/accessibility/AccessibilityNodeObject.cpp:
(WebCore::AccessibilityNodeObject::alternativeText const):
(WebCore::AccessibilityNodeObject::visibleText const):
(WebCore::AccessibilityNodeObject::accessibilityText const):
(WebCore::AccessibilityNodeObject::ariaLabeledByText const):
* Source/WebCore/accessibility/AccessibilitySVGObject.cpp:
(WebCore::AccessibilitySVGObject::accessibilityText const):
* Source/WebCore/accessibility/AccessibilityTable.cpp:
(WebCore::AccessibilityTable::labelText const):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to