New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Regression(259658@main) Geolocation permission prompt is no longer showing in modal view #12489
Regression(259658@main) Geolocation permission prompt is no longer showing in modal view #12489
Conversation
e7d960f
to
891735d
Compare
EWS run on current version of this PR (hash 891735d) |
EWS run on previous version of this PR (hash e7d960f) |
@@ -196,7 +196,7 @@ - (void)_executeNextChallenge | |||
[alert addAction:denyAction]; | |||
[alert addAction:allowAction]; | |||
|
|||
[[_activeChallenge->view window].rootViewController presentViewController:alert animated:YES completion:nil]; | |||
[[UIViewController _viewControllerForFullScreenPresentationFromView:_activeChallenge->view.get()] presentViewController:alert animated:YES completion:nil]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is what we do throughout WK2 so π
β¦owing in modal view https://bugs.webkit.org/show_bug.cgi?id=255133 rdar://107352115 Reviewed by Wenson Hsieh and Tim Horton. Use the same view controller to present the prompt as the UIWebView code used to. I have verified locally that this fixes the issue. * Source/WebKit/UIProcess/ios/WKWebGeolocationPolicyDeciderIOS.mm: (-[WKWebGeolocationPolicyDecider _executeNextChallenge]): Canonical link: https://commits.webkit.org/262697@main
891735d
to
14f4676
Compare
Committed 262697@main (14f4676): https://commits.webkit.org/262697@main Reviewed commits have been landed. Closing PR #12489 and removing active labels. |
14f4676
891735d
π iosπ wincairoπ ios-simπ§ͺ wpe-wk2π§ͺ ios-wk2π§ͺ ios-wk2-wptπ§ͺ gtk-wk2π§ͺ api-iosπ§ͺ api-gtkπ§ͺ mac-AS-debug-wk2