[webkit-changes] [WebKit/WebKit] f8e706: Fix media/video-ended-does-not-hold-sleep-assertio...

Dana Estra noreply at github.com
Tue Feb 6 10:54:45 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f8e706aae35a22a265f9f8b01a585400f46a748f
      https://github.com/WebKit/WebKit/commit/f8e706aae35a22a265f9f8b01a585400f46a748f
  Author: Dana Estra <destra at apple.com>
  Date:   2024-02-06 (Tue, 06 Feb 2024)

  Changed paths:
    M Source/WebCore/platform/ios/PlaybackSessionInterfaceAVKit.h
    M Source/WebCore/platform/ios/PlaybackSessionInterfaceAVKit.mm
    M Source/WebCore/platform/ios/PlaybackSessionInterfaceIOS.h
    M Source/WebCore/platform/ios/PlaybackSessionInterfaceIOS.mm
    M Source/WebCore/platform/ios/PlaybackSessionInterfaceLMK.h
    M Source/WebCore/platform/ios/PlaybackSessionInterfaceLMK.mm

  Log Message:
  -----------
  Fix media/video-ended-does-not-hold-sleep-assertion.html iOS layout test crash
https://bugs.webkit.org/show_bug.cgi?id=268782
rdar://122342671

Reviewed by Eric Carlson.

Before, destruction logic for PlaybackSessionInterfaceAVKit/LMK was split up between
Their own destructors and the base class PlayBackSessionInterfaceIOS destructor. This
Patch moves all the logic to the derived classes to avoid a crash that was caused
By a virtual function being called indirectly from the base class's destructor.

* Source/WebCore/platform/ios/PlaybackSessionInterfaceAVKit.h:
* Source/WebCore/platform/ios/PlaybackSessionInterfaceAVKit.mm:
(WebCore::PlaybackSessionInterfaceAVKit::invalidate):
* Source/WebCore/platform/ios/PlaybackSessionInterfaceIOS.h:
* Source/WebCore/platform/ios/PlaybackSessionInterfaceIOS.mm:
(WebCore::PlaybackSessionInterfaceIOS::~PlaybackSessionInterfaceIOS):
(WebCore::PlaybackSessionInterfaceIOS::invalidate): Deleted.
* Source/WebCore/platform/ios/PlaybackSessionInterfaceLMK.h:
* Source/WebCore/platform/ios/PlaybackSessionInterfaceLMK.mm:
(WebCore::PlaybackSessionInterfaceLMK::~PlaybackSessionInterfaceLMK):
(WebCore::PlaybackSessionInterfaceLMK::invalidate):

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




More information about the webkit-changes mailing list