-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Upstream visionOS fullscreen transition logic #15195
Conversation
EWS run on current version of this PR (hash 4adb8ca) |
|
||
auto completion = makeBlockPtr([controller = retainPtr(controller), inWindow = retainPtr(inWindow), originalState = retainPtr(originalState), enter, completionHandler = WTFMove(completionHandler)] (BOOL finished) mutable { | ||
if (!finished) | ||
return; |
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.
(Just curious β is it intended to skip the completionHandler
in the case of !finished
?)
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.
Not intentional... I should remove this check as the animation isn't intended to be interruptible. Probably better in a follow-up, to avoid mixing with upstreaming.
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.
Yep! Definitely good to keep this upstreaming patch as-is.
https://bugs.webkit.org/show_bug.cgi?id=258404 rdar://111164029 Reviewed by Wenson Hsieh. * Source/WebKit/Platform/spi/visionos/MRUIKitSPI.h: Added. * Source/WebKit/Platform/spi/visionos/RealitySimulationServicesSPI.h: Added. * Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm: (WebKit::useSpatialFullScreenTransition): (WebKit::resizeScene): (-[WKFixedSizeWindow _shouldResizeWithScene]): (-[WKFullScreenParentWindowState initWithWindow:]): (-[WKFullscreenWindowSceneDelegate initWithController:originalDelegate:]): (-[WKFullscreenWindowSceneDelegate originalDelegate]): (-[WKFullscreenWindowSceneDelegate windowScene:shouldCloseForReason:]): (-[WKFullscreenWindowSceneDelegate methodSignatureForSelector:]): (-[WKFullscreenWindowSceneDelegate respondsToSelector:]): (-[WKFullscreenWindowSceneDelegate forwardInvocation:]): (-[WKFullscreenWindowSceneDelegate forwardingTargetForSelector:]): (-[WKFullScreenWindowController enterFullScreen:]): (-[WKFullScreenWindowController beganEnterFullScreenWithInitialFrame:finalFrame:]): (-[WKFullScreenWindowController _completedExitFullScreen]): (-[WKFullScreenWindowController _exitFullscreenImmediately]): (-[WKFullScreenWindowController _dismissFullscreenViewController]): (-[WKFullScreenWindowController _configureSpatialFullScreenTransition]): (-[WKFullScreenWindowController _performSpatialFullScreenTransition:completionHandler:]): * Source/WebKit/WebKit.xcodeproj/project.pbxproj: Canonical link: https://commits.webkit.org/265416@main
4adb8ca
to
9866948
Compare
Committed 265416@main (9866948): https://commits.webkit.org/265416@main Reviewed commits have been landed. Closing PR #15195 and removing active labels. |
9866948
4adb8ca
π§ͺ ios-wk2π§ͺ api-macπ§ͺ gtk-wk2π§ͺ mac-AS-debug-wk2