Skip to content

Fix video-suppress-hdr-fullscreen.html using correct is-mac test#48582

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
squelart:eng/fix-video-suppress-hdr-fullscreen-test
Jul 30, 2025
Merged

Fix video-suppress-hdr-fullscreen.html using correct is-mac test#48582
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
squelart:eng/fix-video-suppress-hdr-fullscreen-test

Conversation

@squelart
Copy link
Contributor

@squelart squelart commented Jul 27, 2025

@squelart squelart self-assigned this Jul 27, 2025
@squelart squelart added the Media Bugs related to the HTML 5 Media elements. label Jul 27, 2025
@squelart squelart force-pushed the eng/fix-video-suppress-hdr-fullscreen-test branch from 38b5ba1 to 90b8fca Compare July 29, 2025 03:54
@squelart squelart added the merge-queue Applied to send a pull request to merge-queue label Jul 30, 2025
https://bugs.webkit.org/show_bug.cgi?id=296545
rdar://156678123

Reviewed by Eric Carlson.

* LayoutTests/media/video-suppress-hdr-fullscreen.html:
The UA string test could return `true` in some non-macOS environments.
`testRunner.isMac` should be more accurate.

* LayoutTests/platform/ipad/TestExpectations:
Re-enable the test.

Canonical link: https://commits.webkit.org/298016@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/fix-video-suppress-hdr-fullscreen-test branch from 90b8fca to 38b085f Compare July 30, 2025 00:45
@webkit-commit-queue
Copy link
Collaborator

Committed 298016@main (38b085f): https://commits.webkit.org/298016@main

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

@webkit-commit-queue webkit-commit-queue merged commit 38b085f into WebKit:main Jul 30, 2025
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jul 30, 2025
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.

4 participants