Bug 41452
Summary: | [Qt] fast/dom/beforeload/remove-video-in-beforeload-listener.html crashy, needs updated Phonon in Qt | ||
---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> |
Component: | Media | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED INVALID | ||
Severity: | Normal | CC: | beidson, benjamin, eric, kling, pnormand |
Priority: | P3 | Keywords: | Qt, QtTriaged |
Version: | 528+ (Nightly build) | ||
Hardware: | All | ||
OS: | All | ||
Bug Depends on: | |||
Bug Blocks: | 79668 |
Csaba Osztrogonác
fast/dom/beforeload/script-before-load-dynamic.html crashes intermittently:
http://build.webkit.org/results/Qt%20Linux%20Release/r62246%20%2814367%29/results.html
It might be a regression, because it wasn't flakey earlier.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Csaba Osztrogonác
I can't reproduce it with WebKitTools/Scripts/run-webkit-tests fast/dom/beforeload/script-before-load-dynamic.html --iterations 1000, but
WebKitTools/Scripts/run-webkit-tests fast/dom/beforeload/ --iterations 1000
Here is the backtrace after segfault:
(gdb) bt
#0 Phonon::Gstreamer::Backend::debugLevel (this=0x8351298) at ../../../3rdparty/phonon/gstreamer/backend.cpp:438
#1 0xf119935e in Phonon::Gstreamer::Backend::handleBusMessage (this=0x821f140, message={m_message = 0xffffa614, m_source = 0x5}) at ../../../3rdparty/phonon/gstreamer/backend.cpp:417
#2 0xf1199411 in Phonon::Gstreamer::Backend::qt_metacall (this=0x821f140, _c=QMetaObject::InvokeMetaMethod, _id=1, _a=0x8215070) at .moc/release-shared/moc_backend.cpp:82
#3 0xf3e1b435 in QMetaObject::metacall (object=0x0, cl=QMetaObject::InvokeMetaMethod, idx=5, argv=0x8215070) at kernel/qmetaobject.cpp:237
#4 0xf3e25a36 in QMetaCallEvent::placeMetaCall (this=0x81de190, object=0x821f140) at kernel/qobject.cpp:561
#5 0xf3e26fc3 in QObject::event (this=0x821f140, e=0x81de190) at kernel/qobject.cpp:1240
#6 0xf40870dc in QApplicationPrivate::notify_helper (this=0x81310c0, receiver=0x821f140, e=0x81de190) at kernel/qapplication.cpp:4300
#7 0xf408db22 in QApplication::notify (this=0xffffae94, receiver=0x821f140, e=0x81de190) at kernel/qapplication.cpp:3704
#8 0xf3e15feb in QCoreApplication::notifyInternal (this=0xffffae94, receiver=0x821f140, event=0x81de190) at kernel/qcoreapplication.cpp:704
#9 0xf3e16f4f in QCoreApplicationPrivate::sendPostedEvents (receiver=0x0, event_type=0, data=0x811b478) at ../../include/QtCore/../../src/corelib/kernel/qcoreapplication.h:215
#10 0xf3e170fd in QCoreApplication::sendPostedEvents (receiver=0x0, event_type=0) at kernel/qcoreapplication.cpp:1238
#11 0xf3e42c5f in postEventSourceDispatch (s=0x812a218) at ../../include/QtCore/../../src/corelib/kernel/qcoreapplication.h:220
#12 0xf33131d8 in g_main_context_dispatch () from /usr/lib/libglib-2.0.so.0
#13 0xf3316873 in ?? () from /usr/lib/libglib-2.0.so.0
#14 0x08121018 in ?? ()
#15 0x00000000 in ?? ()
Csaba Osztrogonác
fast/dom/beforeload/remove-video-in-beforeload-listener.html
is added in http://trac.webkit.org/changeset/61744 and it make
fast/dom/beforeload/script-before-load-dynamic.html crash
$ WebKitTools/Scripts/run-webkit-tests fast/dom/beforeload/remove-video-in-beforeload-listener.html fast/dom/beforeload/script-before-load-dynamic.html --debug --iterations 100
182 test cases (91%) succeeded
18 test cases (9%) crashed
If I put fast/dom/beforeload/remove-video-in-beforeload-listener.html into
the Skipped list, other tests never crash with this:
$ WebKitTools/Scripts/run-webkit-tests fast/dom/beforeload --debug --iterations 1000
Csaba Osztrogonác
fast/dom/beforeload/remove-video-in-beforeload-listener.html skipped until fix by http://trac.webkit.org/changeset/62833
Eric Seidel (no email)
I wonder if this could be related to bug 41871. Probably not.
Benjamin Poulain
Should this really be a P1?
Andreas Kling
Taking.
Andreas Kling
This is a bug in the Phonon GStreamer backend, which won't let you delete MediaObjects when you feel like it. It's already fixed upstream on projects.kde.org, and should be downstreamed into Qt before the next release.
Leaving this bug open to track the skipped test.
Philippe Normand
The Phonon backend is no longer used by QtWebKit on Linux.