[webkit-changes] [WebKit/WebKit] ee46c7: [iOS] Make eager soft-linking of Data Detection fr...
Brent Fulgham
noreply at github.com
Wed Apr 19 16:29:39 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: ee46c752943bd6be0e433b8e7931efd5de456e77
https://github.com/WebKit/WebKit/commit/ee46c752943bd6be0e433b8e7931efd5de456e77
Author: Brent Fulgham <bfulgham at apple.com>
Date: 2023-04-19 (Wed, 19 Apr 2023)
Changed paths:
M Source/WebCore/PAL/PAL.xcodeproj/project.pbxproj
M Source/WebCore/PAL/pal/spi/ios/DataDetectorsUISPI.h
A Source/WebCore/PAL/pal/spi/ios/DataDetectorsUISoftLink.h
A Source/WebCore/PAL/pal/spi/ios/DataDetectorsUISoftLink.mm
M Source/WebCore/editing/cocoa/DataDetection.mm
M Source/WebKit/UIProcess/ios/WKActionSheetAssistant.mm
M Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm
M Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm
Log Message:
-----------
[iOS] Make eager soft-linking of Data Detection frameworks non-fatal
https://bugs.webkit.org/show_bug.cgi?id=255369
<rdar://107752677>
Reviewed by David Kilzer.
Eager soft-linking of the Data Detection framework was added in Bug 241267.
Crash telemetry indicates that the soft-linking fails in some cases, possibly
when multiple code paths are trying to initialize the DataDetectors framework,
and one of those paths fails
Instead, we should consolidate the SoftLink code into a single implementation
file, and use that single code path in all cases.
* Source/WebCore/PAL/PAL.xcodeproj/project.pbxproj:
* Source/WebCore/PAL/pal/spi/ios/DataDetectorsUISPI.h: Moved some declarations
to DataDetectorsUISoftLink.
* Source/WebCore/PAL/pal/spi/ios/DataDetectorsUISoftLink.h: Added.
* Source/WebCore/PAL/pal/spi/ios/DataDetectorsUISoftLink.mm: Added.
* Source/WebCore/editing/cocoa/DataDetection.mm: Reference new SoftLink calls.
* Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm: Ditto.
* Source/WebKit/WebProcess/cocoa/WebProcessCocoa.mm:
(WebKit::softlinkDataDetectorsFrameworks): Ditto.
Canonical link: https://commits.webkit.org/263154@main
More information about the webkit-changes
mailing list