RESOLVED FIXED 197117
Add more _WKElementActionTypes and provide API to create with custom types 
https://bugs.webkit.org/show_bug.cgi?id=197117
Summary Add more _WKElementActionTypes and provide API to create with custom types
Dean Jackson
Reported 2019-04-19 14:28:09 PDT
Add more _WKElementActionTypes and provide API to create with custom types
Attachments
Patch (4.35 KB, patch)
2019-04-19 14:34 PDT, Dean Jackson 		no flags
DetailsFormatted DiffDiff
Patch (4.42 KB, patch)
2019-04-19 14:42 PDT, Dean Jackson 		no flags
DetailsFormatted DiffDiff
Patch (3.76 KB, patch)
2019-04-19 14:57 PDT, Dean Jackson 		thorton: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2019-04-19 14:28:36 PDT
<rdar://problem/50059548>
Dean Jackson
Comment 2 2019-04-19 14:34:10 PDT
Created attachment 367828 [details] Patch
Dean Jackson
Comment 3 2019-04-19 14:42:42 PDT
Created attachment 367831 [details] Patch
Tim Horton
Comment 4 2019-04-19 14:44:16 PDT
Comment on attachment 367831 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=367831&action=review > Source/WebKit/UIProcess/API/Cocoa/_WKElementAction.h:45 > + _WKElementActionTypeShare WK_API_AVAILABLE(macos(WK_MAC_TBA), ios(10.0)), This one shouldn't be TBA > Source/WebKit/UIProcess/API/Cocoa/_WKElementAction.h:55 > -+ (instancetype)elementActionWithType:(_WKElementActionType)type customTitle:(NSString *)title; > ++ (instancetype)elementActionWithType:(_WKElementActionType)type title:(NSString *)title actionHandler:(WKElementActionHandler)actionHandler; Are you sure there are no clients of the old SPI‽ Also, this needs availability.
Dean Jackson
Comment 5 2019-04-19 14:57:41 PDT
Created attachment 367835 [details] Patch
Dean Jackson
Comment 6 2019-04-19 15:25:10 PDT
Committed r244472: <https://trac.webkit.org/changeset/244472>
Note You need to log in before you can comment on or make changes to this bug.