WKWebViewContentProvider shouldn't be a UIScrollViewDelegate
<rdar://problem/38967492>
Created attachment 336694 [details] Patch
Comment on attachment 336694 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=336694&action=review > Source/WebKit/UIProcess/Cocoa/WKWebViewContentProvider.h:45 > @protocol WKWebViewContentProvider <NSObject, UIScrollViewDelegate> ‽‽‽ still a UIScrollViewDelegate
Comment on attachment 336694 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=336694&action=review > Source/WebKit/UIProcess/Cocoa/WKWebViewContentProvider.h:47 > - (instancetype)web_initWithFrame:(CGRect) frame webView:(WKWebView *)webView; It’s a little weirder now that the forwarding of scrollViewDidScroll is magical. Probably should be web_scrollViewDidScroll and just be a normal part of the WKWebViewContentProvider protocol.
Created attachment 336695 [details] Patch
Created attachment 336700 [details] Patch
Committed r230041: <https://trac.webkit.org/changeset/230041>
Comment on attachment 336700 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=336700&action=review > Source/WebKit/UIProcess/API/Cocoa/WKWebView.mm:2432 > + [_customContentView web_scrollViewDidScroll:(UIScrollView *)scrollView]; Why was the cast needed? Is it still needed?