[iOS] premierleague.com enters video fullscreen from element fullscre…

…en on iPhone

https://bugs.webkit.org/show_bug.cgi?id=270329
rdar://123721211

Reviewed by Eric Carlson.

premierleague.com does not include a `playsinline` attribute on their video elements,
which causes them to attempt to enter video fullscreen when playing, even in element
fullscreen. Exempt premierleague.com from the `playsinline` attribute (and consider
removing that requirement on iPhone generally).

* Source/WebCore/html/MediaElementSession.cpp:
(WebCore::MediaElementSession::requiresFullscreenForVideoPlayback const):
* Source/WebCore/page/Quirks.cpp:
(WebCore::Quirks::shouldIgnorePlaysInlineRequirementQuirk const):
* Source/WebCore/page/Quirks.h:

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

Source/WebCore/html/MediaElementSession.cpp

@@ -885,6 +885,9 @@ bool MediaElementSession::requiresFullscreenForVideoPlayback() const
     if (!m_element.document().settings().inlineMediaPlaybackRequiresPlaysInlineAttribute())
         return false;
 
+    if (m_element.document().quirks().shouldIgnorePlaysInlineRequirementQuirk())
+        return false;
+
 #if PLATFORM(IOS_FAMILY)
     if (CocoaApplication::isIBooks())
         return !m_element.hasAttributeWithoutSynchronization(HTMLNames::webkit_playsinlineAttr) && !m_element.hasAttributeWithoutSynchronization(HTMLNames::playsinlineAttr);

Source/WebCore/page/Quirks.cpp

@@ -1819,4 +1819,22 @@ bool Quirks::needsIpadMiniUserAgent(StringView host)
     return false;
 }
 
+
+bool Quirks::shouldIgnorePlaysInlineRequirementQuirk() const
+{
+#if PLATFORM(IOS_FAMILY)
+    if (!needsQuirks())
+        return false;
+
+    if (m_shouldIgnorePlaysInlineRequirementQuirk)
+        return *m_shouldIgnorePlaysInlineRequirementQuirk;
+
+    m_shouldIgnorePlaysInlineRequirementQuirk = isDomain("premierleague.com"_s);
+
+    return *m_shouldIgnorePlaysInlineRequirementQuirk;
+#else
+    return false;
+#endif
+}
+
 }

Source/WebCore/page/Quirks.h

@@ -186,6 +186,7 @@ class Quirks {
     bool needsDisableDOMPasteAccessQuirk() const;
 
     bool shouldDisableElementFullscreenQuirk() const;
+    bool shouldIgnorePlaysInlineRequirementQuirk() const;
 
 private:
     bool needsQuirks() const;
@@ -257,6 +258,7 @@ class Quirks {
     mutable std::optional<bool> m_shouldDisableDataURLPaddingValidation;
     mutable std::optional<bool> m_needsDisableDOMPasteAccessQuirk;
     mutable std::optional<bool> m_shouldDisableElementFullscreen;
+    mutable std::optional<bool> m_shouldIgnorePlaysInlineRequirementQuirk;
 
     Vector<RegistrableDomain> m_subFrameDomainsForStorageAccessQuirk;
 };