Bug 225332 – REGRESSION (r276883): [Debug] ASSERTION FAILED: index != notFound in WebCore::PlatformMediaSessionManager::setCurrentSession()

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[Truitt Savell]

imported/w3c/web-platform-tests/webaudio/the-audio-api/the-audiocontext-interface/suspend-after-construct.html

This test began crashing with the changes in https://trac.webkit.org/changeset/276883/webkit

History:
imported/w3c/web-platform-tests/webaudio/the-audio-api/the-audiocontext-interface/suspend-after-construct.html

Log;
Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   com.apple.JavaScriptCore      	0x000000065a2a73be WTFCrash + 14 (Assertions.cpp:305)
1   com.apple.WebCore             	0x0000000639610eeb WTFCrashWithInfo(int, char const*, char const*, int) + 27 (Assertions.h:695)
2   com.apple.WebCore             	0x000000063dab1dd5 WebCore::PlatformMediaSessionManager::setCurrentSession(WebCore::PlatformMediaSession&) + 293 (PlatformMediaSessionManager.cpp:312)
3   com.apple.WebCore             	0x000000063b1764b7 WebCore::MediaSessionManagerCocoa::setCurrentSession(WebCore::PlatformMediaSession&) + 39 (MediaSessionManagerCocoa.mm:213)

https://build.webkit.org/results/Apple-BigSur-Debug-WK2-Tests/r276920%20(1605)/imported/w3c/web-platform-tests/webaudio/the-audio-api/the-audiocontext-interface/suspend-after-construct-crash-log.txt

[Radar WebKit Bug Importer]

<rdar://problem/77476145>

[Ryan Haddad]

Created attachment 427616 [details]
crash log

ASSERTION FAILED: index != notFound
./platform/audio/PlatformMediaSessionManager.cpp(312) : virtual void WebCore::PlatformMediaSessionManager::setCurrentSession(WebCore::PlatformMediaSession &)

[Jer Noble]

Yep, caused by r276883. I'm super curious how this wasn't hit by the EWS bots though.

[Jer Noble]

The session is setActive(true) immediately after the ASSERT. Looks like we need to move up the activation by a couple lines.

[Jer Noble]

Created attachment 427672 [details]
Patch

[EWS]

Committed r276978 (237305@main): <https://commits.webkit.org/237305@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 427672 [details].