[webkit-changes] [WebKit/WebKit] 1c010c: Make WeakPtr to Nodes thread safe again

Chris Dumez noreply at github.com
Mon Jun 17 16:27:17 PDT 2024


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

  Changed paths:
    M Source/WebCore/dom/EventTarget.h

  Log Message:
  -----------
  Make WeakPtr to Nodes thread safe again
https://bugs.webkit.org/show_bug.cgi?id=275575

Reviewed by Andres Gonzalez.

Make WeakPtr to Nodes thread safe again. This shows as performance neutral on the benchmarks
we track. The idea was to make the WeakPtrImpl ref count thread safe so that we could use
WeakPtr<Node> in more places, where performance is critical. However, this didn't work out
and we plan to use CheckedPtr<Node> for those cases instead.

Note that being able to pass WeakPtr<Node> to another thread is important for accessibility
code as well.

* Source/WebCore/dom/EventTarget.h:

Canonical link: https://commits.webkit.org/280097@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