[webkit-changes] [WebKit/WebKit] e7d4b6: Remove WebFullScreenManagerProxy::supportsFullScreen

Alex Christensen noreply at github.com
Tue Jan 21 13:24:07 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: e7d4b610f7854eb95aba3b9bd32c65914bd76a0a
      https://github.com/WebKit/WebKit/commit/e7d4b610f7854eb95aba3b9bd32c65914bd76a0a
  Author: Alex Christensen <achristensen at apple.com>
  Date:   2025-01-21 (Tue, 21 Jan 2025)

  Changed paths:
    M Source/WebKit/UIProcess/WebFullScreenManagerProxy.cpp
    M Source/WebKit/UIProcess/WebFullScreenManagerProxy.h
    M Source/WebKit/UIProcess/WebFullScreenManagerProxy.messages.in
    M Source/WebKit/WebProcess/InjectedBundle/InjectedBundlePageFullScreenClient.cpp

  Log Message:
  -----------
  Remove WebFullScreenManagerProxy::supportsFullScreen
https://bugs.webkit.org/show_bug.cgi?id=285874
rdar://142836497

Reviewed by Tim Horton.

There's no reason to send a synchronous IPC to get a value you know will be true
on non-iOS platforms and a value from a bool you just sent on iOS.
This change should only increase the responsiveness during a fullscreen transition.

* Source/WebKit/UIProcess/WebFullScreenManagerProxy.cpp:
(WebKit::WebFullScreenManagerProxy::supportsFullScreen):
* Source/WebKit/UIProcess/WebFullScreenManagerProxy.h:
* Source/WebKit/UIProcess/WebFullScreenManagerProxy.messages.in:
* Source/WebKit/WebProcess/InjectedBundle/InjectedBundlePageFullScreenClient.cpp:
(WebKit::InjectedBundlePageFullScreenClient::supportsFullScreen):

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



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list