[webkit-changes] [WebKit/WebKit] 644d65: Highlight appears behind dictated text

mwyrzykowski noreply at github.com
Fri Jun 9 12:32:31 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 644d6535804c8d7a1ad1204e4203ba2462acf0a8
      https://github.com/WebKit/WebKit/commit/644d6535804c8d7a1ad1204e4203ba2462acf0a8
  Author: Mike Wyrzykowski <mwyrzykowski at apple.com>
  Date:   2023-06-09 (Fri, 09 Jun 2023)

  Changed paths:
    M Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfiguration.mm
    M Source/WebKit/UIProcess/mac/WebViewImpl.mm

  Log Message:
  -----------
  Highlight appears behind dictated text
https://bugs.webkit.org/show_bug.cgi?id=257874
radar://110492195

Reviewed by Aditya Keerthi.

264903 at main introduced a regression where the following code:

  auto initialUnderlines = extractInitialUnderlines(string);
  underlines = !initialUnderlines.isEmpty() ? initialUnderlines : extractUnderlines(string);

was changed to:
  underlines = compositionUnderlines(string);

but compositionUnderlines doesn't populate the underlines when they are otherwise empty.

Re-introduce code removed from 264903 at main and this issue appears resolved.

* Source/WebKit/UIProcess/API/Cocoa/WKWebViewConfiguration.mm:
(-[WKWebViewConfiguration setAllowsInlinePredictions:]):
(-[WKWebViewConfiguration allowsInlinePredictions]):
* Source/WebKit/UIProcess/mac/WebViewImpl.mm:
(WebKit::extractUnderlines):
(WebKit::WebViewImpl::setMarkedText):

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




More information about the webkit-changes mailing list