Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 8e1031fc9a3ad0d3705c50866a49a019e7370363
https://github.com/WebKit/WebKit/commit/8e1031fc9a3ad0d3705c50866a49a019e7370363
Author: Richard Robinson <[email protected]>
Date: 2024-02-05 (Mon, 05 Feb 2024)
Changed paths:
A
LayoutTests/editing/spelling/document-markers-remain-after-removing-text-expected.txt
A
LayoutTests/editing/spelling/document-markers-remain-after-removing-text.html
M Source/WebCore/dom/DocumentMarkerController.cpp
Log Message:
-----------
Document markers sometimes get erroneously removed on earlier words when
removing text in a later part of the sentence
https://bugs.webkit.org/show_bug.cgi?id=268778
rdar://122336557
Reviewed by Megan Gardner and Aditya Keerthi.
When a text node (for example, "AAAA BBBB CCCC") has a document marker on it
(for example, a spelling
marker on "AAAA"), and some text is then removed from it (like "CCCC"), the
document marker will get
removed accidentally.
This is because the logic in `shiftMarkers` does not guard against markers that
occur in the prior
part of the text, which should just be completely ignored and not removed or
shifted, since they
would not be affected by the removal of later text.
Fix by checking for such a case and not removing the marker in that case.
*
LayoutTests/editing/spelling/document-markers-remain-after-removing-text.html:
Added.
*
LayoutTests/platform/ios-simulator/editing/spelling/document-markers-remain-after-removing-text-expected.txt:
Added.
* Source/WebCore/dom/DocumentMarkerController.cpp:
(WebCore::DocumentMarkerController::shiftMarkers):
Canonical link: https://commits.webkit.org/274129@main
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
