Bug 213142

Summary: Don't claim to support fullscreen mode unless fullScreenEnabled setting is enabled
Product: WebKit Reporter: Eric Carlson <eric.carlson>
Component: MediaAssignee: Eric Carlson <eric.carlson>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, changseok, esprehn+autocc, ews-watchlist, glenn, gyuyoung.kim, jer.noble, philipj, rackler, sergio, tsavell, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch for landing.
none
Patch for landing
none
Patch for landing
none
Patch for landing
none
Patch for landing
none
Patch none

Description Eric Carlson 2020-06-12 13:17:50 PDT
Don't claim to support fullscreen mode unless fullScreenEnabled setting is enabled

<rdar://problem/63753327>
Comment 1 Eric Carlson 2020-06-12 15:06:50 PDT
Created attachment 401791 [details]
Patch
Comment 2 Eric Carlson 2020-06-12 15:50:04 PDT
Created attachment 401797 [details]
Patch for landing.
Comment 3 Eric Carlson 2020-06-12 16:32:54 PDT
Created attachment 401800 [details]
Patch for landing
Comment 4 Eric Carlson 2020-06-15 10:19:50 PDT
Created attachment 401908 [details]
Patch for landing
Comment 5 Eric Carlson 2020-06-15 11:48:14 PDT
Created attachment 401917 [details]
Patch for landing
Comment 6 Eric Carlson 2020-06-15 15:25:42 PDT
Created attachment 401948 [details]
Patch for landing
Comment 7 Eric Carlson 2020-06-16 13:11:55 PDT
Created attachment 402034 [details]
Patch
Comment 8 EWS 2020-06-16 15:41:23 PDT
Committed r263121: <https://trac.webkit.org/changeset/263121>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 402034 [details].
Comment 9 Karl Rackler 2020-06-19 14:46:20 PDT
It looks like the changes in this submit broke this test:
media/video-fullscreen-only-playback.html 

Tracking this failure in:
https://bugs.webkit.org/show_bug.cgi?id=213407
Comment 10 Truitt Savell 2020-06-19 17:11:37 PDT
Reverted r263121 for reason:

Broke media/video-fullscreen-only-playback.html on Catalina Debug

Committed r263301: <https://trac.webkit.org/changeset/263301>
Comment 11 EWS 2020-06-24 08:55:03 PDT
Committed r263458: <https://trac.webkit.org/changeset/263458>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 402034 [details].