-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Unified Text Replacement] Refactor UnifiedTextReplacementController methods to pass in a session type instead of separate parameters #28760
Conversation
EWS run on previous version of this PR (hash 7ea52e3) |
using ReplacementType = WebUnifiedTextReplacementSessionDataReplacementType; | ||
|
||
WTF::UUID uuid; | ||
ReplacementType replacementType; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit - might be good to specify an explicit initial value here to prevent UB.
EWS run on current version of this PR (hash e6e3fd3) |
β¦methods to pass in a session type instead of separate parameters https://bugs.webkit.org/show_bug.cgi?id=274356 rdar://128333023 Reviewed by Wenson Hsieh. Work towards using more strongly typed objects instead of passing around UUIDs everywhere by creating a new `WebUnifiedTextReplacementSessionData` type and using that, similar to the existing `WebUnifiedTextReplacementContextData` type. This also has the advantage of being more expressive with regards to optional-ness. * Source/WebKit/Scripts/webkit/messages.py: (headers_for_type): * Source/WebKit/Shared/UnifiedTextReplacement.serialization.in: * Source/WebKit/Shared/WebUnifiedTextReplacementContextData.h: * Source/WebKit/Shared/WebUnifiedTextReplacementSessionData.h: Copied from Source/WebKit/Shared/WebUnifiedTextReplacementContextData.h. * Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm: (WebKit::WebPageProxy::willBeginTextReplacementSession): (WebKit::WebPageProxy::didBeginTextReplacementSession): (WebKit::WebPageProxy::textReplacementSessionDidReceiveReplacements): (WebKit::WebPageProxy::textReplacementSessionDidUpdateStateForReplacement): (WebKit::WebPageProxy::didEndTextReplacementSession): (WebKit::WebPageProxy::textReplacementSessionDidReceiveTextWithReplacementRange): (WebKit::WebPageProxy::textReplacementSessionDidReceiveEditAction): (WebKit::WebPageProxy::addTextIndicatorStyleForID): * Source/WebKit/UIProcess/WebPageProxy.h: * Source/WebKit/WebKit.xcodeproj/project.pbxproj: * Source/WebKit/WebProcess/WebPage/Cocoa/UnifiedTextReplacementController.mm: (WebKit::UnifiedTextReplacementController::willBeginTextReplacementSession): (WebKit::UnifiedTextReplacementController::didBeginTextReplacementSession): (WebKit::UnifiedTextReplacementController::textReplacementSessionDidReceiveReplacements): (WebKit::UnifiedTextReplacementController::textReplacementSessionDidUpdateStateForReplacement): (WebKit::UnifiedTextReplacementController::didEndTextReplacementSession<WebUnifiedTextReplacementSessionData::ReplacementType::PlainText>): (WebKit::UnifiedTextReplacementController::didEndTextReplacementSession<WebUnifiedTextReplacementSessionData::ReplacementType::RichText>): (WebKit::UnifiedTextReplacementController::didEndTextReplacementSession): (WebKit::UnifiedTextReplacementController::textReplacementSessionDidReceiveTextWithReplacementRange): (WebKit::UnifiedTextReplacementController::textReplacementSessionDidReceiveEditAction): (WebKit::UnifiedTextReplacementController::textReplacementSessionPerformEditActionForPlainText): (WebKit::UnifiedTextReplacementController::textReplacementSessionPerformEditActionForRichText): (WebKit::UnifiedTextReplacementController::didEndTextReplacementSession<WebUnifiedTextReplacementType::PlainText>): Deleted. (WebKit::UnifiedTextReplacementController::didEndTextReplacementSession<WebUnifiedTextReplacementType::RichText>): Deleted. * Source/WebKit/WebProcess/WebPage/Cocoa/WebPageCocoa.mm: (WebKit::WebPage::updateTextIndicatorStyleVisibilityForID): (WebKit::WebPage::willBeginTextReplacementSession): (WebKit::WebPage::didBeginTextReplacementSession): (WebKit::WebPage::textReplacementSessionDidReceiveReplacements): (WebKit::WebPage::textReplacementSessionDidUpdateStateForReplacement): (WebKit::WebPage::didEndTextReplacementSession): (WebKit::WebPage::textReplacementSessionDidReceiveTextWithReplacementRange): (WebKit::WebPage::textReplacementSessionDidReceiveEditAction): * Source/WebKit/WebProcess/WebPage/UnifiedTextReplacementController.h: * Source/WebKit/WebProcess/WebPage/WebPage.h: * Source/WebKit/WebProcess/WebPage/WebPage.messages.in: Canonical link: https://commits.webkit.org/278965@main
e6e3fd3
to
670f9c7
Compare
Committed 278965@main (670f9c7): https://commits.webkit.org/278965@main Reviewed commits have been landed. Closing PR #28760 and removing active labels. |
670f9c7
e6e3fd3
π wincairoπ ios-simπ mac-AS-debugπ§ͺ wpe-wk2π§ͺ wincairo-testsπ§ͺ webkitperlπ§ͺ ios-wk2π§ͺ api-macπ§ͺ api-wpeπ§ͺ ios-wk2-wptπ§ͺ api-iosπ§ͺ mac-wk2π tvπ§ͺ mac-AS-debug-wk2π§ͺ gtk-wk2π§ͺ mac-wk2-stressπ§ͺ api-gtkπ watch