Started to enable encrypted media tests on the GTK port, the following tests are failing in a similar backtrace to this one since r237091 (not a related revision, just the trunk version I tested a debug build with) Thread 1 (Thread 0x7f38e77fe700 (LWP 38379)): #0 _g_log_abort () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gmessages.c:554 #1 0x00007f39c4eb2c1b in g_logv () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gmessages.c:1362 #2 0x00007f39c4eb2d6f in g_log () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gmessages.c:1403 #3 0x00007f39c4eccd0e in g_strv_length () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gstrfuncs.c:2896 #4 0x00007f391058e18a in gst_qtdemux_request_protection_context () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gst-plugins-good-1.14.4/gst/isomp4/qtdemux.c:7993 #5 gst_qtdemux_configure_protected_caps () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gst-plugins-good-1.14.4/gst/isomp4/qtdemux.c:8081 #6 gst_qtdemux_configure_stream () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gst-plugins-good-1.14.4/gst/isomp4/qtdemux.c:8312 #7 0x00007f3910590903 in gst_qtdemux_add_stream () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gst-plugins-good-1.14.4/gst/isomp4/qtdemux.c:8461 #8 qtdemux_expose_streams () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gst-plugins-good-1.14.4/gst/isomp4/qtdemux.c:12336 #9 0x00007f39105ad3f8 in gst_qtdemux_process_adapter () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gst-plugins-good-1.14.4/gst/isomp4/qtdemux.c:6780 #10 0x00007f39be3eb8eb in gst_pad_chain_data_unchecked () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gstreamer-1.14.4/gst/gstpad.c:4322 #11 gst_pad_push_data () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gstreamer-1.14.4/gst/gstpad.c:4578 #12 0x00007f39be3f3c13 in gst_pad_push () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gstreamer-1.14.4/gst/gstpad.c:4697 #13 0x00007f39be6ee43d in gst_base_src_loop () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gstreamer-1.14.4/libs/gst/base/gstbasesrc.c:2957 #14 0x00007f39be4202a9 in gst_task_func () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/gstreamer-1.14.4/gst/gsttask.c:332 #15 0x00007f39c4ed4140 in g_thread_pool_thread_proxy () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gthreadpool.c:307 #16 0x00007f39c4ed3775 in g_thread_proxy () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gthread.c:784 #17 0x00007f39bfb566db in start_thread (arg=0x7f38e77fe700) at pthread_create.c:463 #18 0x00007f39c74e988f in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95 Failing tests, imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-destroy-persistent-license.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-persistent-license-events.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-persistent-license.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-retrieve-destroy-persistent-license.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-retrieve-persistent-license.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-clear-encrypted.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-encrypted-clear-sources.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-encrypted-clear.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-multikey.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-playback-temporary-setMediaKeys-immediately.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-setMediaKeys-onencrypted.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-two-videos.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-waitingforkey.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-reset-src-after-setmediakeys.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-again-after-playback.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-again-after-resetting-src.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-multiple-times-with-different-mediakeys.https.html [ Crash ] imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-waiting-for-a-key.https.html
Created attachment 352314 [details] Patch
Created attachment 352315 [details] Patch
Comment on attachment 352315 [details] Patch EME is not enabled in GTK, why would we want to do this?
(In reply to Xabier Rodríguez Calvar from comment #3) > Comment on attachment 352315 [details] > Patch > > EME is not enabled in GTK, why would we want to do this? It's not enabled because the feature has not been developed. Why would be not want EME enabled in GTK? It's a convenient port to develop from the desktop. First step is having test expectations so that fixes can be sensibly proposed. Not the first time, https://bugs.webkit.org/show_bug.cgi?id=99024
(In reply to Charlie Turner from comment #4) > It's not enabled because the feature has not been developed. Why would be > not want EME enabled in GTK? It's a convenient port to develop from the > desktop. First step is having test expectations so that fixes can be > sensibly proposed. > > Not the first time, https://bugs.webkit.org/show_bug.cgi?id=99024 Even when I agree with you, I'll let Michael tell you. Maybe we can make him change his mind.
Comment on attachment 352315 [details] Patch This patch just unskips tests and improves the test expectations, so it seems uncontroversial to me.
Comment on attachment 352315 [details] Patch Clearing flags on attachment: 352315 Committed r237180: <https://trac.webkit.org/changeset/237180>
All reviewed patches have been landed. Closing bug.