RESOLVED FIXED 236806
Refactor share menu item presentation 
https://bugs.webkit.org/show_bug.cgi?id=236806
Summary Refactor share menu item presentation
Kate Cheney
Reported 2022-02-17 14:46:32 PST
Refactor share menu item presentation
Attachments
Patch (5.41 KB, patch)
2022-02-17 15:02 PST, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Patch (5.34 KB, patch)
2022-02-17 15:55 PST, Kate Cheney 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Kate Cheney
Comment 1 2022-02-17 15:02:02 PST
Created attachment 452436 [details] Patch
Kate Cheney
Comment 2 2022-02-17 15:03:31 PST Comment hidden (obsolete)
Kate Cheney
Comment 3 2022-02-17 15:04:20 PST
rdar://problem/88858606
Aditya Keerthi
Comment 4 2022-02-17 15:20:45 PST
Comment on attachment 452436 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452436&action=review > Source/WebCore/PAL/pal/spi/mac/NSSharingServicePickerSPI.h:31 > +@interface NSSharingServicePicker (Internal) Nit: "Staging" is a more accurate name for the category. > Source/WebCore/PAL/pal/spi/mac/NSSharingServicePickerSPI.h:-44 > -- (void)getMenuWithCompletion:(void(^)(NSMenu *))completion; I think this deletion will cause open source builds to fail, `getMenuWithCompletion:` is still used in `WebContextMenuProxyMac`.
Megan Gardner
Comment 5 2022-02-17 15:21:50 PST
Comment on attachment 452436 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452436&action=review > Source/WebCore/PAL/pal/spi/mac/NSSharingServicePickerSPI.h:32 > +- (NSMenuItem *)standardShareMenuItem; Staging_rdarNumber to be specific
Kate Cheney
Comment 6 2022-02-17 15:23:23 PST
Comment on attachment 452436 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=452436&action=review thanks for the review! >> Source/WebCore/PAL/pal/spi/mac/NSSharingServicePickerSPI.h:32 >> +- (NSMenuItem *)standardShareMenuItem; > > Staging_rdarNumber to be specific will fix! >> Source/WebCore/PAL/pal/spi/mac/NSSharingServicePickerSPI.h:-44 >> -- (void)getMenuWithCompletion:(void(^)(NSMenu *))completion; > > I think this deletion will cause open source builds to fail, `getMenuWithCompletion:` is still used in `WebContextMenuProxyMac`. Ah you're right, this is leftover from a previous iteration.
Kate Cheney
Comment 7 2022-02-17 15:55:22 PST
Created attachment 452442 [details] Patch
EWS
Comment 8 2022-02-17 18:32:34 PST
Committed r290103 (247449@main): <https://commits.webkit.org/247449@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 452442 [details].
Note You need to log in before you can comment on or make changes to this bug.