Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 616bcfacc22d680d4303f938c0ca25666f79e0ef
https://github.com/WebKit/WebKit/commit/616bcfacc22d680d4303f938c0ca25666f79e0ef
Author: Andres Gonzalez <[email protected]>
Date: 2023-04-18 (Tue, 18 Apr 2023)
Changed paths:
M Source/WebCore/accessibility/AccessibilityObject.h
M Source/WebCore/accessibility/isolatedtree/mac/AXIsolatedObjectMac.mm
M Source/WebCore/accessibility/mac/AccessibilityObjectMac.mm
M Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperBase.h
Log Message:
-----------
AX: Spell check attributes for an isolated object's subrange do not match the
spell check attributes of the object's full range.
https://bugs.webkit.org/show_bug.cgi?id=255574
<rdar://problem/108174222>
Reviewed by Chris Fleizach.
We were caching the attributed string for an isolated object range with spell
check attributes and using it for any subrange. However, the spell check
attributes of the full range may not match the spell check attributes of a
subrange. For instance, if the object's range text is "hello world", there is
no misspellings in it. But if a subrange with offset 1 and length 9 is
requested, this would contains the text "ello worl" which contains two
misspelled words. For this reason, it is necessary to perform the spell check
upon request of subranges.
Covered by test accessibility/content-editable-as-textarea.html.
* Source/WebCore/accessibility/AccessibilityObject.h:
* Source/WebCore/accessibility/isolatedtree/mac/AXIsolatedObjectMac.mm:
(WebCore::AXIsolatedObject::initializePlatformProperties):
(WebCore::AXIsolatedObject::cachedAttributedStringForTextMarkerRange const):
* Source/WebCore/accessibility/mac/AccessibilityObjectMac.mm:
(WebCore::attributedStringSetSpelling):
* Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperBase.h:
Canonical link: https://commits.webkit.org/263070@main
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
