[webkit-changes] [WebKit/WebKit] 5d4cdb: [visionOS] System chrome autohides while resizing ...
Aditya Keerthi
noreply at github.com
Tue Sep 5 23:00:16 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 5d4cdbee487c4fdd0a88d1425f36595dc692a923
https://github.com/WebKit/WebKit/commit/5d4cdbee487c4fdd0a88d1425f36595dc692a923
Author: Aditya Keerthi <akeerthi at apple.com>
Date: 2023-09-05 (Tue, 05 Sep 2023)
Changed paths:
M Source/WebKit/Configurations/WebKit.xcconfig
M Source/WebKit/Platform/spi/ios/UIKitSPI.h
M Source/WebKit/Platform/spi/visionos/MRUIKitSPI.h
M Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenViewController.mm
Log Message:
-----------
[visionOS] System chrome autohides while resizing or repositioning fullscreen video
https://bugs.webkit.org/show_bug.cgi?id=261176
rdar://114793136
Reviewed by Tim Horton.
266191 at main made system chrome autohide alongside WebKit's controls in fullscreen.
However, autohide is currently not prevented while the user is interacting with
system chrome, resulting in a poor experience.
To fix, listen for notifications that indicate system chrome is being used. If
it is, reset the autohide timer.
* Source/WebKit/Configurations/WebKit.xcconfig:
Link MRUIKit, as the scene positioning notification comes from there.
* Source/WebKit/Platform/spi/ios/UIKitSPI.h:
* Source/WebKit/Platform/spi/visionos/MRUIKitSPI.h:
* Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenViewController.mm:
Drive-by: Drop 'm_' from an Objective-C instance variable.
(-[WKFullScreenViewController initWithWebView:]):
(-[WKFullScreenViewController hideUI]):
(-[WKFullScreenViewController videoControlsManagerDidChange]):
(-[WKFullScreenViewController hideCustomControls:]):
(-[WKFullScreenViewController _didBeginInteractionWithSystemChrome:]):
(-[WKFullScreenViewController _didEndInteractionWithSystemChrome:]):
Canonical link: https://commits.webkit.org/267661@main
More information about the webkit-changes
mailing list