Skip to content

Try to fix the tvOS build after 293779@main#44224

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
aestes:eng/Try-to-fix-the-tvOS-build-after-293779-main
Apr 17, 2025
Merged

Try to fix the tvOS build after 293779@main#44224
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
aestes:eng/Try-to-fix-the-tvOS-build-after-293779-main

Conversation

@aestes
Copy link
Contributor

@aestes aestes commented Apr 17, 2025

03557b4

Try to fix the tvOS build after 293779@main
https://bugs.webkit.org/show_bug.cgi?id=291712

Unreviewed build fix; called createNSString() instead of relying on implicit WTFString->NSString
conversion. While here, fixed a comment in WTFString.h that claimed we still support this type of
implicit conversion.

* Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.mm:
(WebCore::MediaSessionManagerCocoa::updateNowPlayingSuppression):

Canonical link: https://commits.webkit.org/293821@main

9543f51

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style 🛠 ios ✅ 🛠 mac 🛠 wpe 🛠 win
✅ 🧪 bindings ✅ 🛠 ios-sim 🛠 mac-AS-debug 🧪 wpe-wk2 🧪 win-tests
✅ 🧪 webkitperl 🧪 ios-wk2 🧪 api-mac 🧪 api-wpe
🧪 ios-wk2-wpt 🧪 mac-wk1 🛠 wpe-cairo
🧪 api-ios 🧪 mac-wk2 🛠 gtk
🛠 vision 🧪 mac-AS-debug-wk2 🧪 gtk-wk2
🛠 vision-sim 🧪 mac-wk2-stress 🧪 api-gtk
⏳ 🧪 vision-wk2 🧪 mac-intel-wk2 loading 🛠 playstation
✅ 🛠 🧪 unsafe-merge 🛠 tv 🛠 mac-safer-cpp
🛠 tv-sim
🛠 watch
🛠 watch-sim

@aestes aestes self-assigned this Apr 17, 2025
@aestes aestes added the Media Bugs related to the HTML 5 Media elements. label Apr 17, 2025
@aestes aestes added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Apr 17, 2025
https://bugs.webkit.org/show_bug.cgi?id=291712

Unreviewed build fix; called createNSString() instead of relying on implicit WTFString->NSString
conversion. While here, fixed a comment in WTFString.h that claimed we still support this type of
implicit conversion.

* Source/WebCore/platform/audio/cocoa/MediaSessionManagerCocoa.mm:
(WebCore::MediaSessionManagerCocoa::updateNowPlayingSuppression):

Canonical link: https://commits.webkit.org/293821@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Try-to-fix-the-tvOS-build-after-293779-main branch from 9543f51 to 03557b4 Compare April 17, 2025 22:03
@webkit-commit-queue
Copy link
Collaborator

Committed 293821@main (03557b4): https://commits.webkit.org/293821@main

Reviewed commits have been landed. Closing PR #44224 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit 03557b4 into WebKit:main Apr 17, 2025
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Apr 17, 2025
@aestes aestes deleted the eng/Try-to-fix-the-tvOS-build-after-293779-main branch August 25, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Media Bugs related to the HTML 5 Media elements.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments