Bug 248714

Summary: Align modern & old codepaths for fullscreen API
Product: WebKit Reporter: Tim Nguyen (:ntim) <ntim>
Component: DOMAssignee: Tim Nguyen (:ntim) <ntim>
Status: NEW ---    
Severity: Normal CC: webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 276855    

Description Tim Nguyen (:ntim) 2022-12-03 10:41:08 PST
The original implementation of the unprefixed API tried to make as little change as possible to the prefixed API.

The only difference between the two methods is the `isFullyActive()` check. Since we successfully unified that check for `Element::requestFullscreen` in https://commits.webkit.org/255551@main, we should also do it for `Document::fullscreenEnabled`.
Comment 1 Radar WebKit Bug Importer 2022-12-03 10:41:17 PST
<rdar://problem/102937855>
Comment 2 Tim Nguyen (:ntim) 2022-12-03 11:14:17 PST
Pull request: https://github.com/WebKit/WebKit/pull/7117