[webkit-changes] [WebKit/WebKit] ca9aa8: LegacyCustomProtocolManager is incorrectly using t...

Chris Dumez noreply at github.com
Thu Dec 21 16:21:26 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ca9aa80579f0a3eab81504e3e8fcb81d75be4ea9
      https://github.com/WebKit/WebKit/commit/ca9aa80579f0a3eab81504e3e8fcb81d75be4ea9
  Author: Chris Dumez <cdumez at apple.com>
  Date:   2023-12-21 (Thu, 21 Dec 2023)

  Changed paths:
    M Source/WebKit/NetworkProcess/CustomProtocols/Cocoa/LegacyCustomProtocolManagerCocoa.mm
    M Source/WebKit/NetworkProcess/CustomProtocols/LegacyCustomProtocolManager.cpp

  Log Message:
  -----------
  LegacyCustomProtocolManager is incorrectly using the NetworkProcess object off the main thread
https://bugs.webkit.org/show_bug.cgi?id=266787

Reviewed by Alex Christensen.

LegacyCustomProtocolManager is incorrectly using the NetworkProcess object off the
main thread. NetworkProcess is only safe to use on the main thread.

Found this the hardware when I tried adding a threading assertion in 272402 at main.

This patch fixes the bug and re-lands the threading assertion.

* Source/WebKit/NetworkProcess/CustomProtocols/Cocoa/LegacyCustomProtocolManagerCocoa.mm:
(-[WKCustomProtocol startLoading]):
(-[WKCustomProtocol stopLoading]):
* Source/WebKit/NetworkProcess/CustomProtocols/LegacyCustomProtocolManager.cpp:
(WebKit::LegacyCustomProtocolManager::protectedNetworkProcess const):

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




More information about the webkit-changes mailing list