Flesh out WKBrowsingContextLoadDelegate a bit
Created attachment 112394 [details] Patch
Comment on attachment 112394 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=112394&action=review > Source/WebKit2/UIProcess/API/mac/WKBrowsingContextController.mm:218 > + WKBrowsingContextController *browsingContext = (WKBrowsingContextController *)clientInfo; > + [browsingContext.loadDelegate browsingContextControllerDidReceiveServerRedirectForProvisionalLoad:browsingContext]; > +} This does not check that the delegate responds to the selector. The code needs to do that, and possibly cache the result as well. > Source/WebKit2/UIProcess/API/mac/WKBrowsingContextController.mm:257 > + WKBrowsingContextController *browsingContext = (WKBrowsingContextController *)clientInfo; > + [browsingContext.loadDelegate browsingContextControllerDidFailLoad:browsingContext withError:(NSError *)cfError.get()]; Ditto.
Created attachment 112415 [details] Patch
Now with a test!
Committed r98399: <http://trac.webkit.org/changeset/98399>