Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 59955e21f6c4839c6f0613a7f3a8007eda03e523
      
https://github.com/WebKit/WebKit/commit/59955e21f6c4839c6f0613a7f3a8007eda03e523
  Author: Jean-Yves Avenard <[email protected]>
  Date:   2025-02-24 (Mon, 24 Feb 2025)

  Changed paths:
    M Source/WebCore/platform/cocoa/EffectiveRateChangedListener.h
    M Source/WebCore/platform/cocoa/EffectiveRateChangedListener.mm

  Log Message:
  -----------
  ASSERTION FAILED: isMainThread()
https://bugs.webkit.org/show_bug.cgi?id=288375
rdar://145483109

Reviewed by Simon Fraser.

Remove assertion added in 289922@main.
It did expose an issue in the original code. The observer could take a reference
on a thread other than the main thread which could make multiple concurrent 
calls to stop().
We make the stop() and destructor operation thread-safe.

* Source/WebCore/platform/cocoa/EffectiveRateChangedListener.h:
* Source/WebCore/platform/cocoa/EffectiveRateChangedListener.mm:
(WebCore::EffectiveRateChangedListener::EffectiveRateChangedListener): Remove 
out of date comment.
(WebCore::EffectiveRateChangedListener::stop):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to