Summary: | konqueror exits silently after loading a webpage | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | S. Burmeister <sven.burmeister> |
Component: | general | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | andresbajotierra, maksim |
Priority: | NOR | ||
Version: | 4.2.0 | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: | |||
Attachments: | patch |
Description
S. Burmeister
2009-02-28 12:12:46 UTC
What is your Qt4 version ? Thanks rpm -qa | grep libqt4 libqt4-x11-4.4.3-13.1 libqt4-4.4.3-13.1 libqt4-sql-mysql-32bit-4.4.3-15.1 libqt4-sql-sqlite-4.4.3-13.1 libqt4-sql-mysql-4.4.3-15.1 libqt4-32bit-4.4.3-13.1 libqt4-sql-4.4.3-13.1 libqt4-x11-32bit-4.4.3-13.1 libqt4-devel-4.4.3-13.1 libqt4-debuginfo-4.4.3-13.1 libqt4-qt3support-4.4.3-13.1 libqt4-sql-32bit-4.4.3-13.1 Created attachment 31829 [details]
patch
Maksim: continuing the discussion from #179359 here as it looks appropriate indeed.
Does the attached looks like what you had in mind?
Can confirm it fixes #182027; this one isn't reproducible here. One question: why the ref/deref? Wound't Qt kill the event if you remove the QObject? Anyway, didn't think of using Qt events myself... For images' load events, BTW, there is code in DocumentImpl --- dispatchImageLoadEvents and such. It is specific, but we may need to generalize it at some point --- e.g. relayout already liked to produce blur/focus events if the focused entry got yanked, and I just made it drop those. I am sure something else may show up eventually, but no hurry just yet. We also need some solution for the nested event loops.. SVN commit 939174 by ggarand: queue DOM resize events sent to subframes until the toplevel frame has finished resizing. This avoids infinite recursion scenarios where javascript would run during the layout of the toplevel frame. Problem acurately diagnosed by SadEagle. BUG: 185790, 182027 M +20 -0 html/html_objectimpl.cpp M +7 -0 html/html_objectimpl.h M +11 -2 khtmlview.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=939174 SVN commit 939480 by ggarand: automatically merged revision 939174: queue DOM resize events sent to subframes until the toplevel frame has finished resizing. This avoids infinite recursion scenarios where javascript would run during the layout of the toplevel frame. Problem acurately diagnosed by SadEagle. BUG: 185790, 182027 M +20 -0 html/html_objectimpl.cpp M +7 -0 html/html_objectimpl.h M +11 -2 khtmlview.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=939480 |