RESOLVED FIXED239636
Add WKNotification and WKWebsiteDataStore SPI for handling click/close of persistent notifications 
https://bugs.webkit.org/show_bug.cgi?id=239636
Summary Add WKNotification and WKWebsiteDataStore SPI for handling click/close of per...
Brady Eidson
Reported 2022-04-21 20:31:38 PDT
Add WKWebsiteDataStore SPI for handling click/close of persistent notifications This is to move us off of the current stop-gap to something actually works for clients.
Attachments
EWS v1 (54.37 KB, patch)
2022-04-21 20:47 PDT, Brady Eidson 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (59.18 KB, patch)
2022-04-22 13:30 PDT, Brady Eidson 		cdumez: review+
ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch for landing v1 (59.11 KB, patch)
2022-04-22 15:37 PDT, Brady Eidson 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
PFL v2 (59.15 KB, patch)
2022-04-22 16:24 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
PFL v3 (50.21 KB, patch)
2022-04-22 18:26 PDT, Brady Eidson 		beidson: review+
beidson: commit-queue-
DetailsFormatted DiffDiff
PFL v4 (50.62 KB, patch)
2022-04-23 09:20 PDT, Brady Eidson 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
THANK YOU FOR THE PAIN UNIFIED BUILD v5 (51.42 KB, patch)
2022-04-23 12:02 PDT, Brady Eidson 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Never had this much of a problem fighting missing includes in unified build v6 (51.87 KB, patch)
2022-04-23 13:20 PDT, Brady Eidson 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Maybe found the source of problems v7 (52.11 KB, patch)
2022-04-23 13:31 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (8) View All Add attachment proposed patch, testcase, etc.
Brady Eidson
Comment 1 2022-04-21 20:47:19 PDT
Created attachment 458109 [details] EWS v1
Brady Eidson
Comment 2 2022-04-22 13:30:24 PDT
Created attachment 458170 [details] Patch
Chris Dumez
Comment 3 2022-04-22 15:00:08 PDT
Comment on attachment 458170 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=458170&action=review > Source/WebCore/Modules/notifications/NotificationDataCocoa.mm:72 > + NotificationData data { title, body, iconURL, tag, language, direction, originString, URL { String { serviceWorkerRegistrationURL } }, *uuid, PAL::SessionID { sessionID.unsignedLongLongValue }, { } }; Return we could just: return { title, body, iconURL, tag, language, direction, originString, URL { String { serviceWorkerRegistrationURL } }, *uuid, PAL::SessionID { sessionID.unsignedLongLongValue }, { } }; > Tools/WebKitTestRunner/cocoa/WebNotificationProviderCocoa.mm:39 > + RetainPtr<NSDictionary> dictionaryRepresentation = adoptNS(WKNotificationCopyDictionaryRepresentation(notification.get())); could use auto.
Brady Eidson
Comment 4 2022-04-22 15:37:51 PDT
Created attachment 458179 [details] Patch for landing v1
Brady Eidson
Comment 5 2022-04-22 16:24:16 PDT
Created attachment 458182 [details] PFL v2
Brady Eidson
Comment 6 2022-04-22 18:26:24 PDT
Created attachment 458192 [details] PFL v3
Brady Eidson
Comment 7 2022-04-23 09:20:31 PDT
Created attachment 458211 [details] PFL v4
Brady Eidson
Comment 8 2022-04-23 12:02:35 PDT
Created attachment 458220 [details] THANK YOU FOR THE PAIN UNIFIED BUILD v5
Brady Eidson
Comment 9 2022-04-23 13:20:42 PDT
Created attachment 458221 [details] Never had this much of a problem fighting missing includes in unified build v6
Brady Eidson
Comment 10 2022-04-23 13:31:58 PDT
Created attachment 458222 [details] Maybe found the source of problems v7
EWS
Comment 11 2022-04-23 15:28:05 PDT
Committed r293296 (249921@main): <https://commits.webkit.org/249921@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 458222 [details].
Radar WebKit Bug Importer
Comment 12 2022-04-23 15:29:14 PDT
<rdar://problem/92222113>
Note You need to log in before you can comment on or make changes to this bug.