* imported/w3c/web-platform-tests/encrypted-media/clearkey-events.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-keystatuses-multiple-sessions.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-keystatuses.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-events.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-multikey-sequential-readyState.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-multikey-sequential.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-multisession.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-setMediaKeys-after-src.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-setMediaKeys-after-update.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-update-disallowed-input.https.html [ Crash ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-destroy-persistent-license.https.html [ Failure ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-retrieve-destroy-persistent-license.https.html [ Failure ] * imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-retrieve-persistent-license.https.html [ Failure ] imported/w3c/web-platform-tests/encrypted-media/clearkey-events.https.html Stacktrace: STDERR: ASSERTION FAILED: m_cdmProxy STDERR: ../../Source/WebCore/platform/encryptedmedia/CDMProxy.cpp(291) : void WebCore::CDMInstanceProxy::mergeKeysFrom(const WebCore::KeyStore&) STDERR: 1 0x7fdf29ae13b5 WTFCrash STDERR: 2 0x7fdf248a1ccd WTF::CrashOnOverflow::overflowed() STDERR: 3 0x7fdf276524d2 WebCore::CDMInstanceProxy::mergeKeysFrom(WebCore::KeyStore const&) STDERR: 4 0x7fdf28576c7c WebCore::CDMInstanceSessionClearKey::updateLicense(WTF::String const&, WebCore::CDMSessionType, WebCore::SharedBuffer const&, WTF::CompletionHandler<void (bool, WTF::Optional<WTF::Vector<std::pair<WTF::Ref<WebCore::SharedBuffer, WTF::DumbPtrTraits<WebCore::SharedBuffer> >, WebCore::CDMKeyStatus>, 0ul, WTF::CrashOnOverflow, 16ul, WTF::FastMalloc> >&&, WTF::Optional<double>&&, WTF::Optional<std::pair<WebCore::CDMMessageType, WTF::Ref<WebCore::SharedBuffer, WTF::DumbPtrTraits<WebCore::SharedBuffer> > > >&&, WebCore::CDMInstanceSession::SuccessValue)>&&) STDERR: 5 0x7fdf2815f0dd STDERR: 6 0x7fdf2816790c STDERR: 7 0x7fdf248d836b WTF::Function<void ()>::operator()() const STDERR: 8 0x7fdf281687a2 WebCore::ActiveDOMObject::queueTaskKeepingObjectAlive<WebCore::MediaKeySession>(WebCore::MediaKeySession&, WebCore::TaskSource, WTF::Function<void ()>&&)::{lambda()#1}::operator()() const STDERR: 9 0x7fdf2816c922 WTF::Detail::CallableWrapper<WebCore::ActiveDOMObject::queueTaskKeepingObjectAlive<WebCore::MediaKeySession>(WebCore::MediaKeySession&, WebCore::TaskSource, WTF::Function<void ()>&&)::{lambda()#1}, void>::call() STDERR: 10 0x7fdf248d836b WTF::Function<void ()>::operator()() const STDERR: 11 0x7fdf26bc232a WebCore::EventLoopFunctionDispatchTask::execute() STDERR: 12 0x7fdf26bc0e24 WebCore::EventLoop::run() STDERR: 13 0x7fdf26ce4743 WebCore::WindowEventLoop::didReachTimeToRun() STDERR: 14 0x7fdf26cf323c void std::__invoke_impl<void, void (WebCore::WindowEventLoop::*&)(), WebCore::WindowEventLoop*&>(std::__invoke_memfun_deref, void (WebCore::WindowEventLoop::*&)(), WebCore::WindowEventLoop*&) STDERR: 15 0x7fdf26cf31b3 std::__invoke_result<void (WebCore::WindowEventLoop::*&)(), WebCore::WindowEventLoop*&>::type std::__invoke<void (WebCore::WindowEventLoop::*&)(), WebCore::WindowEventLoop*&>(void (WebCore::WindowEventLoop::*&)(), WebCore::WindowEventLoop*&) STDERR: 16 0x7fdf26cf312e void std::_Bind<void (WebCore::WindowEventLoop::*(WebCore::WindowEventLoop*))()>::__call<void, , 0ul>(std::tuple<>&&, std::_Index_tuple<0ul>) STDERR: 17 0x7fdf26cf30e4 void std::_Bind<void (WebCore::WindowEventLoop::*(WebCore::WindowEventLoop*))()>::operator()<, void>() STDERR: 18 0x7fdf26cf30be WTF::Detail::CallableWrapper<std::_Bind<void (WebCore::WindowEventLoop::*(WebCore::WindowEventLoop*))()>, void>::call() STDERR: 19 0x7fdf248d836b WTF::Function<void ()>::operator()() const STDERR: 20 0x7fdf24cecc34 WebCore::Timer::fired() STDERR: 21 0x7fdf27607b93 WebCore::ThreadTimers::sharedTimerFiredInternal() STDERR: 22 0x7fdf276075a7 STDERR: 23 0x7fdf2760c090 STDERR: 24 0x7fdf248d836b WTF::Function<void ()>::operator()() const STDERR: 25 0x7fdf275dddc0 WebCore::MainThreadSharedTimer::fired() STDERR: 26 0x7fdf275e48e2 WTF::RunLoop::Timer<WebCore::MainThreadSharedTimer>::fired() STDERR: 27 0x7fdf29b94691 STDERR: 28 0x7fdf29b946e9 STDERR: 29 0x7fdf29b93dbc STDERR: 30 0x7fdf29b93dec STDERR: 31 0x7fdf13c006b8 g_main_context_dispatch STDERR: LEAK: 1 WebPageProxy More stacktraces: https://build.webkit.org/results/WPE%20Linux%2064-bit%20Debug%20(Tests)/r256504%20(3471)/results.html
The reason these assertions were not caught during development is that I work with the GTK port on the desktop, and these imported W3C tests are not enabled there, which they should have been. There's no need for different expectations between this ports for this feature at least. Separate issue now. The reason for the failures is that the W3C tests do not use the setMediaKeys() API to associate the key session with the player. The spec does say applications are supposed to handle implementations that can not continue key session algorithms without this association, but nowadays goes on to concede that such implementations are "uncommon"... Most real-world players and indeed other tests simply ensure they call setMediaKeys() after the key session is created for this reason. In our architecture, this is important since cdmInstanceAttached() is used internally as a sync point for other setup steps that are prerequisites for key session algorithms. I'll think more about a workaround, a very simple solution is to fix the W3C tests, but they also consider us uncommon, so it'd be nice to be a common implementation instead.
I decided the best choice is to skip these tests for WPE as well as GTK to give me time to correctly triage them. There are new passes, crashes and timeouts in equal measure, so this is not removing any useful testing.
Created attachment 390759 [details] Patch
Created attachment 390777 [details] Patch for landing
Comment on attachment 390777 [details] Patch for landing Clearing flags on attachment: 390777 Committed r256622: <https://trac.webkit.org/changeset/256622>
All reviewed patches have been landed. Closing bug.