Add UIDelegatePrivate SPI corresponding to WKPageUIClient.showPage
Created attachment 318703 [details] Patch
Comment on attachment 318703 [details] Patch r=me
Comment on attachment 318703 [details] Patch Clearing flags on attachment: 318703 Committed r221026: <http://trac.webkit.org/changeset/221026>
All reviewed patches have been landed. Closing bug.
<rdar://problem/34016741>
This caused an api timeout with the test WebKit2.ShowPage on iOS Simulators https://build.webkit.org/builders/Apple%20iOS%2010%20Simulator%20Release%20WK2%20%28Tests%29/builds/3781/steps/run-api-tests/logs/stdio
Reverted r221026 for reason: This caused a consistent API failure on iOS Simulator. Committed r221047: <http://trac.webkit.org/changeset/221047>
Created attachment 318802 [details] Patch
Created attachment 318804 [details] Patch
http://trac.webkit.org/r221055
What's up with the icky name?
_webViewClose corresponds with WKPageUIClient.close. I figured _webViewShow was a good name for a function that corresponds with WKPageUIClient.showPage. All the other methods in UIDelegatePrivate start with _webView...
(In reply to Alex Christensen from comment #12) > _webViewClose corresponds with WKPageUIClient.close. I figured _webViewShow > was a good name for a function that corresponds with > WKPageUIClient.showPage. You figured incorrectly :) It should at a /minimum/ be -(void)_showWebView:(WKWebView *)webView. Please fix! > All the other methods in UIDelegatePrivate start > with _webView... Not true.