[webkit-changes] [WebKit/WebKit] a534e0: Ensure that there is always a MainThreadPermission...

Rupin Mittal noreply at github.com
Thu Sep 15 09:09:00 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a534e0313e6ee5c09e2a5f843dd19d644247811c
      https://github.com/WebKit/WebKit/commit/a534e0313e6ee5c09e2a5f843dd19d644247811c
  Author: Rupin Mittal <rmittal2 at apple.com>
  Date:   2022-09-15 (Thu, 15 Sep 2022)

  Changed paths:
    M Source/WebCore/Modules/permissions/MainThreadPermissionObserver.cpp
    M Source/WebCore/Modules/permissions/MainThreadPermissionObserver.h
    M Source/WebCore/Modules/permissions/PermissionStatus.cpp
    M Source/WebCore/Modules/permissions/PermissionStatus.h

  Log Message:
  -----------
  Ensure that there is always a MainThreadPermissionObserver
https://bugs.webkit.org/show_bug.cgi?id=245201

Reviewed by Chris Dumez.

Previously, PermissionStatus was a type of PermissionObserver and an
associated MainThreadPermissionObserver was created only when the
PermissionStatus object was created on a worker thread. This patch
changes PermissionStatus so that it is no longer a type of PermissionObserver
and a associated MainThreadPermissionObserver is always created, regardless
of it the PermissionStatus object was created on the main thread or a
worker thread.

* Source/WebCore/Modules/permissions/MainThreadPermissionObserver.cpp:
(WebCore::MainThreadPermissionObserver::MainThreadPermissionObserver):
(WebCore::MainThreadPermissionObserver::stateChanged):
* Source/WebCore/Modules/permissions/MainThreadPermissionObserver.h:
* Source/WebCore/Modules/permissions/PermissionStatus.cpp:
(WebCore::PermissionStatus::PermissionStatus):
* Source/WebCore/Modules/permissions/PermissionStatus.h:

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




More information about the webkit-changes mailing list