[webkit-changes] [WebKit/WebKit] cbd5d8: REGRESSION (262850 at main): [ iOS, macOS Debug ] Tes...

Chris Dumez noreply at github.com
Mon Jun 12 13:48:24 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: cbd5d8089b51367c195902b841af603705a01544
      https://github.com/WebKit/WebKit/commit/cbd5d8089b51367c195902b841af603705a01544
  Author: Chris Dumez <cdumez at apple.com>
  Date:   2023-06-12 (Mon, 12 Jun 2023)

  Changed paths:
    M Source/WebKit/Platform/IPC/JSIPCBinding.h
    M Tools/TestWebKitAPI/Tests/WebKitCocoa/IPCTestingAPI.mm

  Log Message:
  -----------
  REGRESSION (262850 at main): [ iOS, macOS Debug ] TestWebKitAPI.IPCTestingAPI.CanInterceptHasStorageAccess is a constant failure.
https://bugs.webkit.org/show_bug.cgi?id=255563
rdar://108168809

Reviewed by Alex Christensen.

In 262850 at main, I modified the jsValueForDecodedArgumentValue() taking an
ObjectIdentifier<U> to take an ObjectIdentifierGenericBase instead, in order
to support AtomicObjectIdentifiers as well. However, this broke the API test,
indicating that the wrong jsValueForDecodedArgumentValue() is likely getting
picked now. To address the issue, I am restoring the ObjectIdentifier<U>
overload and introducing a new AtomicObjectIdentifier<U> instead.

* Source/WebKit/Platform/IPC/JSIPCBinding.h:
(IPC::jsValueForDecodedArgumentValue):
* Tools/TestWebKitAPI/Tests/WebKitCocoa/IPCTestingAPI.mm:

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




More information about the webkit-changes mailing list