[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