Skip to content

Conversation

eric-carlson
Copy link
Contributor

@eric-carlson eric-carlson commented Feb 16, 2023

6db3bff

REGRESSION(259626@main): Fullscreen and PiP buttons don't work in a media document
https://bugs.webkit.org/show_bug.cgi?id=252417
rdar://105469592

Reviewed by Jer Noble.

* Source/WebCore/html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::enterFullscreen): Consume a transient activation if fullscreen
is required, but don't reject is there is no activation because this method can not be
called unless the element is allowed to enter fullscreen.

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

e249153

Misc iOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 wincairo
✅ 🧪 bindings ✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🛠 gtk
✅ 🧪 webkitperl 🧪 ios-wk2 ✅ 🧪 api-mac 🧪 gtk-wk2
🧪 api-ios ✅ 🧪 mac-wk1 🧪 api-gtk
✅ 🛠 tv 🧪 mac-wk2
✅ 🛠 tv-sim ✅ 🧪 mac-AS-debug-wk2
✅ 🛠 watch ✅ 🧪 mac-wk2-stress
✅ 🛠 🧪 merge ✅ 🛠 watch-sim

@eric-carlson eric-carlson self-assigned this Feb 16, 2023
@eric-carlson eric-carlson added the Media Bugs related to the HTML 5 Media elements. label Feb 16, 2023
@jyavenard
Copy link
Member

  • but don't reject is there
  • but don't reject it there

@eric-carlson eric-carlson added the merge-queue Applied to send a pull request to merge-queue label Feb 16, 2023
…edia document

https://bugs.webkit.org/show_bug.cgi?id=252417
rdar://105469592

Reviewed by Jer Noble.

* Source/WebCore/html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::enterFullscreen): Consume a transient activation if fullscreen
is required, but don't reject is there is no activation because this method can not be
called unless the element is allowed to enter fullscreen.

Canonical link: https://commits.webkit.org/260401@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the fix-fullscreen-transient-activation branch from e249153 to 6db3bff Compare February 16, 2023 23:39
@webkit-early-warning-system webkit-early-warning-system merged commit 6db3bff into WebKit:main Feb 16, 2023
@webkit-commit-queue
Copy link
Collaborator

Committed 260401@main (6db3bff): https://commits.webkit.org/260401@main

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

@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Feb 16, 2023
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.

5 participants