RESOLVED FIXED 206398
Add WKScriptMessageHandler API that asynchronously responds with a promise 
https://bugs.webkit.org/show_bug.cgi?id=206398
Summary Add WKScriptMessageHandler API that asynchronously responds with a promise
Brady Eidson
Reported 2020-01-16 22:05:29 PST
Add WKScriptMessageHandler API that asynchronously responds with a promise
Attachments
Patch (71.98 KB, patch)
2020-04-18 23:53 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Patch (73.19 KB, patch)
2020-04-19 01:17 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Patch (75.84 KB, patch)
2020-04-19 09:27 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Patch (76.04 KB, patch)
2020-04-19 09:45 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Patch (77.62 KB, patch)
2020-04-19 11:05 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Patch (79.76 KB, patch)
2020-04-19 11:40 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Patch (79.81 KB, patch)
2020-04-19 12:01 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Patch (omg maybe this is the one) (79.82 KB, patch)
2020-04-19 12:15 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
PFR (80.51 KB, patch)
2020-04-19 13:28 PDT, Brady Eidson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (8) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2020-01-16 22:05:46 PST
<rdar://problem/58671599>
Brady Eidson
Comment 2 2020-04-18 14:40:09 PDT
(This is actually at rdar://problem/57243492)
Brady Eidson
Comment 3 2020-04-18 16:24:20 PDT
Blocked on https://bugs.webkit.org/show_bug.cgi?id=210700 as we'll be adding new actual *API* that uses WKContentWorld, and therefore the bug described over in will become very real for more people
Brady Eidson
Comment 4 2020-04-18 23:53:22 PDT
Created attachment 396895 [details] Patch
Brady Eidson
Comment 5 2020-04-19 01:17:23 PDT
Created attachment 396898 [details] Patch While I whittle away at the GTK/etc build failures, Mac and iOS are good to go. Let's review!
EWS Watchlist
Comment 6 2020-04-19 01:18:10 PDT
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Brady Eidson
Comment 7 2020-04-19 09:27:40 PDT
Created attachment 396914 [details] Patch
Brady Eidson
Comment 8 2020-04-19 09:45:57 PDT
Created attachment 396915 [details] Patch
Brady Eidson
Comment 9 2020-04-19 11:05:31 PDT
Created attachment 396918 [details] Patch
Brady Eidson
Comment 10 2020-04-19 11:40:45 PDT
Created attachment 396919 [details] Patch
Brady Eidson
Comment 11 2020-04-19 12:01:44 PDT
Created attachment 396921 [details] Patch
Brady Eidson
Comment 12 2020-04-19 12:15:54 PDT
Created attachment 396923 [details] Patch (omg maybe this is the one)
Brady Eidson
Comment 13 2020-04-19 13:22:56 PDT
(In reply to Brady Eidson from comment #12) > Created attachment 396923 [details] > Patch (omg maybe this is the one) It was. Updating ChangeLogs
Brady Eidson
Comment 14 2020-04-19 13:28:15 PDT
Created attachment 396925 [details] PFR
EWS
Comment 15 2020-04-19 22:55:17 PDT
Committed r260350: <https://trac.webkit.org/changeset/260350> All reviewed patches have been landed. Closing bug and clearing flags on attachment 396925 [details].
Note You need to log in before you can comment on or make changes to this bug.