Skip to content

Commit

Permalink
Update to Vision idiom.
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=259907
rdar://113531570

Reviewed by Wenson Hsieh.

Updated to new idiom now that reality idiom is deprecated.

* Source/WebCore/platform/ios/Device.cpp:
(WebCore::deviceClassIsVision):
(WebCore::deviceClassIsReality): Deleted.
* Source/WebCore/platform/ios/Device.h:
* Source/WebCore/platform/ios/LocalCurrentTraitCollection.mm:
(WebCore::adjustedTraitCollection):
* Source/WebKit/Shared/UserInterfaceIdiom.h:
* Source/WebKit/Shared/UserInterfaceIdiom.mm:
(WebKit::currentUserInterfaceIdiomIsVision):
(WebKit::updateCurrentUserInterfaceIdiom):
(WebKit::currentUserInterfaceIdiomIsReality): Deleted.
* Source/WebKit/Shared/UserInterfaceIdiom.serialization.in:
* Source/WebKit/Shared/WebPreferencesDefaultValues.cpp:
(WebKit::defaultAlternateFormControlDesignEnabled):
(WebKit::defaultVideoFullscreenRequiresElementFullscreen):
* Source/WebKit/UIProcess/Cocoa/WKShareSheet.mm:
(-[WKShareSheet presentWithShareDataArray:inRect:]):
* Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm:
(WebKit::useSpatialFullScreenTransition):

Canonical link: https://commits.webkit.org/266673@main
  • Loading branch information
megangardner committed Aug 8, 2023
1 parent bc52720 commit 9776a6b
Show file tree
Hide file tree
Showing 9 changed files with 18 additions and 18 deletions.
2 changes: 1 addition & 1 deletion Source/WebCore/platform/ios/Device.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ bool deviceClassIsSmallScreen()
return deviceClass == MGDeviceClassiPhone || deviceClass == MGDeviceClassiPod || deviceClass == MGDeviceClassWatch;
}

bool deviceClassIsReality()
bool deviceClassIsVision()
{
#if PLATFORM(VISION)
static auto deviceClass = MGGetSInt32Answer(kMGQDeviceClassNumber, MGDeviceClassInvalid);
Expand Down
2 changes: 1 addition & 1 deletion Source/WebCore/platform/ios/Device.h
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ String deviceName(); // Thread-safe.
// If a check like this is needed, often currentUserInterfaceIdiomIsSmallScreen is preferred.
WEBCORE_EXPORT bool deviceClassIsSmallScreen();

WEBCORE_EXPORT bool deviceClassIsReality();
WEBCORE_EXPORT bool deviceClassIsVision();

// FIXME: How does this differ from !deviceClassIsSmallScreen()?
WEBCORE_EXPORT bool deviceHasIPadCapability();
Expand Down
6 changes: 3 additions & 3 deletions Source/WebCore/platform/ios/LocalCurrentTraitCollection.mm
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,9 @@
static UITraitCollection *adjustedTraitCollection(UITraitCollection *traitCollection)
{
#if PLATFORM(VISION)
// Use the iPad idiom instead of the Reality idiom, since some system colors are transparent
// in the Reality idiom, and are not web-compatible.
if (traitCollection.userInterfaceIdiom == UIUserInterfaceIdiomReality)
// Use the iPad idiom instead of the Vision idiom, since some system colors are transparent
// in the Vision idiom, and are not web-compatible.
if (traitCollection.userInterfaceIdiom == UIUserInterfaceIdiomVision)
return [PAL::getUITraitCollectionClass() traitCollectionWithTraitsFromCollections:@[ traitCollection, [PAL::getUITraitCollectionClass() traitCollectionWithUserInterfaceIdiom:UIUserInterfaceIdiomPad] ]];
#endif
return traitCollection;
Expand Down
4 changes: 2 additions & 2 deletions Source/WebKit/Shared/UserInterfaceIdiom.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,11 @@ namespace WebKit {
enum class UserInterfaceIdiom : uint8_t {
Default,
SmallScreen,
Reality
Vision
};

bool currentUserInterfaceIdiomIsSmallScreen();
bool currentUserInterfaceIdiomIsReality();
bool currentUserInterfaceIdiomIsVision();

UserInterfaceIdiom currentUserInterfaceIdiom();
void setCurrentUserInterfaceIdiom(UserInterfaceIdiom);
Expand Down
12 changes: 6 additions & 6 deletions Source/WebKit/Shared/UserInterfaceIdiom.mm
Original file line number Diff line number Diff line change
Expand Up @@ -42,11 +42,11 @@ bool currentUserInterfaceIdiomIsSmallScreen()
return s_currentUserInterfaceIdiom == UserInterfaceIdiom::SmallScreen;
}

bool currentUserInterfaceIdiomIsReality()
bool currentUserInterfaceIdiomIsVision()
{
if (!s_currentUserInterfaceIdiom)
updateCurrentUserInterfaceIdiom();
return s_currentUserInterfaceIdiom == UserInterfaceIdiom::Reality;
return s_currentUserInterfaceIdiom == UserInterfaceIdiom::Vision;
}

UserInterfaceIdiom currentUserInterfaceIdiom()
Expand All @@ -72,15 +72,15 @@ bool updateCurrentUserInterfaceIdiom()
if (![UIApplication sharedApplication]) {
if (WebCore::deviceClassIsSmallScreen())
return UserInterfaceIdiom::SmallScreen;
if (WebCore::deviceClassIsReality())
return UserInterfaceIdiom::Reality;
if (WebCore::deviceClassIsVision())
return UserInterfaceIdiom::Vision;
} else {
auto idiom = [[UIDevice currentDevice] userInterfaceIdiom];
if (idiom == UIUserInterfaceIdiomPhone || idiom == UIUserInterfaceIdiomWatch)
return UserInterfaceIdiom::SmallScreen;
#if PLATFORM(VISION)
if (idiom == UIUserInterfaceIdiomReality)
return UserInterfaceIdiom::Reality;
if (idiom == UIUserInterfaceIdiomVision)
return UserInterfaceIdiom::Vision;
#endif
}

Expand Down
2 changes: 1 addition & 1 deletion Source/WebKit/Shared/UserInterfaceIdiom.serialization.in
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
enum class WebKit::UserInterfaceIdiom : uint8_t {
Default,
SmallScreen,
Reality
Vision
};

#endif
4 changes: 2 additions & 2 deletions Source/WebKit/Shared/WebPreferencesDefaultValues.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -70,12 +70,12 @@ bool defaultShouldPrintBackgrounds()

bool defaultAlternateFormControlDesignEnabled()
{
return currentUserInterfaceIdiomIsReality();
return currentUserInterfaceIdiomIsVision();
}

bool defaultVideoFullscreenRequiresElementFullscreen()
{
return currentUserInterfaceIdiomIsReality();
return currentUserInterfaceIdiomIsVision();
}

#endif
Expand Down
2 changes: 1 addition & 1 deletion Source/WebKit/UIProcess/Cocoa/WKShareSheet.mm
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,7 @@ - (void)presentWithShareDataArray:(NSArray *)sharingItems inRect:(std::optional<
}];

#if PLATFORM(VISION)
if (webView.traitCollection.userInterfaceIdiom == UIUserInterfaceIdiomReality) {
if (webView.traitCollection.userInterfaceIdiom == UIUserInterfaceIdiomVision) {
[_shareSheetViewController setAllowsCustomPresentationStyle:YES];
[_shareSheetViewController setModalPresentationStyle:UIModalPresentationFormSheet];
} else
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ static void replaceViewWithView(UIView *view, UIView *otherView)

static bool useSpatialFullScreenTransition()
{
return [[UIDevice currentDevice] userInterfaceIdiom] == UIUserInterfaceIdiomReality;
return [[UIDevice currentDevice] userInterfaceIdiom] == UIUserInterfaceIdiomVision;
}

static void resizeScene(UIWindowScene *scene, CGSize size, CompletionHandler<void()>&& completionHandler)
Expand Down

0 comments on commit 9776a6b

Please sign in to comment.