-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Fix internal visionOS build. #41061
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
Fix internal visionOS build. #41061
Conversation
|
EWS run on previous version of this PR (hash 7119450) Details |
|
Commit message contains (OOPS!) and no valid reviewer found, blocking PR #41061. Details: Build #14387 |
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.
Please fix indentation and use FIXME: (all caps).
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.
Ideally something like:
// FIXME: Stop using `AVAudioSessionCategoryOptionAllowBluetooth` as it is deprecated (rdar://145294046).
7119450 to
59d9a75
Compare
|
EWS run on previous version of this PR (hash 59d9a75) Details |
59d9a75 to
0bd7851
Compare
|
EWS run on current version of this PR (hash 0bd7851) Details |
https://bugs.webkit.org/show_bug.cgi?id=288197 rdar://145293550 Reviewed by Chris Dumez. Fix internal visionOS build. * Source/WebCore/platform/audio/ios/AudioSessionIOS.mm: (WebCore::AudioSessionIOS::setCategory): Canonical link: https://commits.webkit.org/290797@main
0bd7851 to
e91baab
Compare
|
Committed 290797@main (e91baab): https://commits.webkit.org/290797@main Reviewed commits have been landed. Closing PR #41061 and removing active labels. |
e91baab
0bd7851
🛠 win🧪 wpe-wk2🧪 win-tests🧪 ios-wk2🧪 api-mac🧪 api-wpe🧪 ios-wk2-wpt🧪 mac-wk1🛠 wpe-cairo🧪 api-ios🧪 mac-wk2🛠 gtk🧪 mac-AS-debug-wk2🧪 gtk-wk2🧪 mac-wk2-stress🧪 api-gtk🧪 vision-wk2🧪 mac-intel-wk2🛠 playstation🛠 mac-safer-cpp