VOOZH about

URL: https://phabricator.services.mozilla.com/D301197

⇱ ⚙ D301197 WIP: Bug 1913666 - Part 13: Wire WindowGlobal IPC trip for fullscreen enter/exit r=edgar!,#dom-core!


WIP: Bug 1913666 - Part 13: Wire WindowGlobal IPC trip for fullscreen enter/exit r=edgar!,#dom-core!
Changes PlannedPublicDraft

Authored by sfarre on May 18 2026, 10:14 PM.
Tags
None
Referenced Files
F72884487: D301197.1781747600.diff
Wed, Jun 17, 1:53 AM
Unknown Object (File)
Mon, Jun 1, 12:27 PM
Unknown Object (File)
Mon, Jun 1, 12:33 AM
Unknown Object (File)
Sun, May 31, 8:49 PM
Unknown Object (File)
Thu, May 28, 10:26 PM
Unknown Object (File)
Wed, May 27, 1:16 AM
Unknown Object (File)
Mon, May 25, 5:36 PM
Unknown Object (File)
Sat, May 23, 11:13 PM
Subscribers

Details

Reviewers
None
Bugzilla Bug ID
1913666
Summary

Three PWindowGlobal IPC messages introduced:

  1. RequestFullscreen - maps to element.requestFullscreen() and queuing a fullscreen request/transaction with the Fullscreen Service
  2. RequestExitFullscreen - maps to document.exitFullscreen() and queuing an exit of fullscreen request/transaction wiith the Fullscreen service
  3. FullscreenServiceTransactionComplete - The IPC message a request uses to complete a transaction with the fullscreen service, which informs the fullscreen service of the final result of the request and then allows for next Fullsreen API operation to start (if any is queued).

This transactional model, is what synchronizes access to fullscreen operations. Elements can still be removed mid-flight of one of these operations, and the behavior is best-effort and exiting if some unexpected result happens.

Diff Detail

Repository
rFIREFOXAUTOLAND firefox-autoland
Branch
HEAD

Event Timeline

sfarre created this revision.
phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".May 18 2026, 10:15 PM
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: secure-revision.
Comment Actions

Code analysis found 1 defect in diff 1277263:

  • 1 defect found by clang-format (Mozlint)
WARNING: Found 1 defect (warning level) that can be dismissed.

You can run this analysis locally with:


If you see a problem in this automated review, please report it here.

You can view these defects in the Diff Detail section of Phabricator diff 1277263.

Revision Contents

CommitTreeParentsAuthorSummaryDate
a913abef564cd9fdf4980611236d3eab96c7Simon Farre
WIP: Bug 1913666 - Part 13: Wire WindowGlobal IPC trip for fullscreen… (Show More…)
May 18 2026, 8:06 PM
StatusAuthorRevision
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre
Changes Plannedsfarre

Diff 1277263

dom/base/Document.cpp

Loading...

dom/base/Element.cpp

Loading...

dom/base/FullscreenService.h

Loading...

dom/base/FullscreenService.cpp

Loading...

dom/ipc/PWindowGlobal.ipdl

Loading...

dom/ipc/WindowGlobalChild.h

Loading...

dom/ipc/WindowGlobalParent.h

Loading...

dom/ipc/WindowGlobalParent.cpp

Loading...