[webkit-changes] [WebKit/WebKit] ce71cf: [Unified Text Replacement] Replacing text with dif...

Richard Robinson noreply at github.com
Thu Feb 29 11:59:10 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ce71cfe36aeaaf258530276b82c1e84ce957b6c4
      https://github.com/WebKit/WebKit/commit/ce71cfe36aeaaf258530276b82c1e84ce957b6c4
  Author: Richard Robinson <richard_robinson2 at apple.com>
  Date:   2024-02-29 (Thu, 29 Feb 2024)

  Changed paths:
    M Source/WebKit/WebProcess/WebPage/UnifiedTextReplacementController.cpp

  Log Message:
  -----------
  [Unified Text Replacement] Replacing text with differing text lengths causes the text to be replaced incorrectly
https://bugs.webkit.org/show_bug.cgi?id=270264
rdar://123790734

Reviewed by Wenson Hsieh and Megan Gardner.

When replacing the text/content, also ensure the corresponding ranges are updated. To facilitate this,
the temporary selection that was being made is no longer temporary, as it never actually needed to be.

Also remove `using namespace WebCore` for more consistency.

* Source/WebKit/WebProcess/WebPage/UnifiedTextReplacementController.cpp:
(WebKit::replaceTextInRange):
(WebKit::replaceContentsInRange):
(WebKit::findReplacementMarkerByUUID):
(WebKit::UnifiedTextReplacementController::textReplacementSessionDidReceiveReplacements):
(WebKit::UnifiedTextReplacementController::textReplacementSessionDidUpdateStateForReplacement):
(WebKit::UnifiedTextReplacementController::didEndTextReplacementSession):
(WebKit::UnifiedTextReplacementController::textReplacementSessionDidReceiveTextWithReplacementRange):
(WebKit::UnifiedTextReplacementController::textReplacementSessionDidReceiveEditAction):

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



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


More information about the webkit-changes mailing list