Bug 82052 - [Qt] REGRESSION(r111480): It made svg/as-image and svg/as-objects tests assert
Summary: [Qt] REGRESSION(r111480): It made svg/as-image and svg/as-objects tests assert
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P1 Critical
Assignee: Tim Horton
URL:
Keywords: Qt, QtTriaged
: 82434 (view as bug list)
Depends on:
Blocks: 78315 79668
  Show dependency treegraph
 
Reported: 2012-03-23 06:48 PDT by Csaba Osztrogonác
Modified: 2012-05-14 03:50 PDT (History)
10 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2012-03-23 06:48:23 PDT
You can easily reproduce it: $Tools/Scripts/old-run-webkit-tests --debug svg/as-image

first run:
svg/as-image/img-preserveAspectRatio-support-2.html -> crashed
svg/as-image/svg-image-leak-loader.html -> crashed

second run:
svg/as-image/animated-svg-as-image.html -> crashed
svg/as-image/svg-image-leak-loader.html -> crashed

...


ASSERTION FAILED: !needsLayout()
../../../../Source/WebCore/page/FrameView.cpp(2958) : virtual void WebCore::FrameView::paintContents(WebCore::GraphicsContext*, const WebCore::IntRect&)
1   0x7f2bc7c28b27 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::FrameView::paintContents(WebCore::GraphicsContext*, WebCore::IntRect const&)+0x25f) [0x7f2bc7c28b27]
2   0x7f2bc7d2b811 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::ScrollView::paint(WebCore::GraphicsContext*, WebCore::IntRect const&)+0x4ab) [0x7f2bc7d2b811]
3   0x7f2bc8155d89 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::SVGImage::draw(WebCore::GraphicsContext*, WebCore::FloatRect const&, WebCore::FloatRect const&, WebCore::ColorSpace, WebCore::CompositeOperator)+0x3a3) [0x7f2bc8155d89]
4   0x7f2bc81558f3 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::SVGImage::drawSVGToImageBuffer(WebCore::ImageBuffer*, WebCore::IntSize const&, float, WebCore::SVGImage::ShouldClearBuffer)+0x321) [0x7f2bc81558f3]
5   0x7f2bc8158bf9 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::SVGImageCache::redraw()+0x13b) [0x7f2bc8158bf9]
6   0x7f2bc8158a87 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::SVGImageCache::imageContentChanged()+0x113) [0x7f2bc8158a87]
7   0x7f2bc7b06e9e /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::CachedImage::changedInRect(WebCore::Image const*, WebCore::IntRect const&)+0x7e) [0x7f2bc7b06e9e]
8   0x7f2bc815490b /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::SVGImageChromeClient::invalidateContentsAndRootView(WebCore::IntRect const&, bool)+0x93) [0x7f2bc815490b]
9   0x7f2bc7bbe94c /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::Chrome::invalidateContentsAndRootView(WebCore::IntRect const&, bool)+0x42) [0x7f2bc7bbe94c]
10  0x7f2bc7d2abc9 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::ScrollView::repaintContentRectangle(WebCore::IntRect const&, bool)+0x19d) [0x7f2bc7d2abc9]
11  0x7f2bc7c2442c /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::FrameView::doDeferredRepaints()+0x19e) [0x7f2bc7c2442c]
12  0x7f2bc7c24184 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::FrameView::endDeferredRepaints()+0x138) [0x7f2bc7c24184]
13  0x7f2bc7c21269 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::FrameView::layout(bool)+0xcbb) [0x7f2bc7c21269]
14  0x7f2bc7c24783 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::FrameView::layoutTimerFired(WebCore::Timer<WebCore::FrameView>*)+0x21) [0x7f2bc7c24783]
15  0x7f2bc7c31518 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::Timer<WebCore::FrameView>::fired()+0x6e) [0x7f2bc7c31518]
16  0x7f2bc7d4a3e2 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::ThreadTimers::sharedTimerFiredInternal()+0xc6) [0x7f2bc7d4a3e2]
17  0x7f2bc7d4a319 /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::ThreadTimers::sharedTimerFired()+0x19) [0x7f2bc7d4a319]
18  0x7f2bc801ad1a /home/oszi/WebKit/WebKitBuild/Debug/lib/libQtWebKit.so.4(WebCore::SharedTimerQt::timerEvent(QTimerEvent*)+0x6a) [0x7f2bc801ad1a]
19  0x7f2bc0c0e519 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(QObject::event(QEvent*)+0xa9) [0x7f2bc0c0e519]
20  0x7f2bc14ca19c /usr/local/Trolltech/Qt-4.8.0/lib/libQtGui.so.4(QApplicationPrivate::notify_helper(QObject*, QEvent*)+0xac) [0x7f2bc14ca19c]
21  0x7f2bc14d106d /usr/local/Trolltech/Qt-4.8.0/lib/libQtGui.so.4(QApplication::notify(QObject*, QEvent*)+0x13d) [0x7f2bc14d106d]
22  0x7f2bc0bfc0ec /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(QCoreApplication::notifyInternal(QObject*, QEvent*)+0x8c) [0x7f2bc0bfc0ec]
23  0x7f2bc0c3052e /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(+0x1cf52e) [0x7f2bc0c3052e]
24  0x7f2bc0c2d47d /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(+0x1cc47d) [0x7f2bc0c2d47d]
25  0x7f2bc2e486f2 /lib/libglib-2.0.so.0(g_main_context_dispatch+0x1f2) [0x7f2bc2e486f2]
26  0x7f2bc2e4c568 /lib/libglib-2.0.so.0(+0x42568) [0x7f2bc2e4c568]
27  0x7f2bc2e4c71c /lib/libglib-2.0.so.0(g_main_context_iteration+0x6c) [0x7f2bc2e4c71c]
28  0x7f2bc0c2d163 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(QEventDispatcherGlib::processEvents(QFlags<QEventLoop::ProcessEventsFlag>)+0x73) [0x7f2bc0c2d163]
29  0x7f2bc1576a9e /usr/local/Trolltech/Qt-4.8.0/lib/libQtGui.so.4(+0x2c2a9e) [0x7f2bc1576a9e]
30  0x7f2bc0bfaca2 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(QEventLoop::processEvents(QFlags<QEventLoop::ProcessEventsFlag>)+0x32) [0x7f2bc0bfaca2]
31  0x7f2bc0bfb114 /usr/local/Trolltech/Qt-4.8.0/lib/libQtCore.so.4(QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>)+0x164) [0x7f2bc0bfb114]
Comment 1 Csaba Osztrogonác 2012-03-23 07:01:24 PDT
Similar bug: https://bugs.webkit.org/show_bug.cgi?id=79631
Comment 2 Csaba Osztrogonác 2012-03-23 07:02:15 PDT
tests skipped - http://trac.webkit.org/changeset/111857
Comment 3 Csaba Osztrogonác 2012-03-28 00:35:08 PDT
*** Bug 82434 has been marked as a duplicate of this bug. ***
Comment 4 Csaba Osztrogonác 2012-03-28 00:36:36 PDT
Tim, Hajime, could you check and fix the regression you caused, please?
Comment 5 Csaba Osztrogonác 2012-03-28 00:37:09 PDT
New failing tests skipped by r112370
Comment 6 Kristóf Kosztyó 2012-03-28 02:57:36 PDT
the svg/as-object tests also start to assert
I skipped them in r112378 http://trac.webkit.org/changeset/112378
Comment 7 Csaba Osztrogonác 2012-03-30 05:12:22 PDT
Set priority to P1, because crashes/asserts are always serious problems. (But I don't think if anybody is interested in fixing bugs ...)
Comment 8 Zoltan Arvai 2012-03-30 06:40:18 PDT
Other asserting test skipped in r112662: 

svg/zoom/page/zoom-mask-with-percentages.svg 
http://trac.webkit.org/changeset/112662
Comment 9 Tim Horton 2012-03-30 10:57:37 PDT
(In reply to comment #7)
> Set priority to P1, because crashes/asserts are always serious problems. (But I don't think if anybody is interested in fixing bugs ...)

I'll take it, since I've seen reports of it happening on Mac too, intermittently. Ugh.
Comment 10 Tim Horton 2012-04-05 11:06:06 PDT
(In reply to comment #9)
> (In reply to comment #7)
> > Set priority to P1, because crashes/asserts are always serious problems. (But I don't think if anybody is interested in fixing bugs ...)
> 
> I'll take it, since I've seen reports of it happening on Mac too, intermittently. Ugh.

Stephen managed to make this reproduce and (thankfully) fixed it!

See if http://trac.webkit.org/changeset/113323 helps.
Comment 11 Csaba Osztrogonác 2012-04-06 03:08:37 PDT
(In reply to comment #10)
> (In reply to comment #9)
> > (In reply to comment #7)
> > > Set priority to P1, because crashes/asserts are always serious problems. (But I don't think if anybody is interested in fixing bugs ...)
> > 
> > I'll take it, since I've seen reports of it happening on Mac too, intermittently. Ugh.
> 
> Stephen managed to make this reproduce and (thankfully) fixed it!
> 
> See if http://trac.webkit.org/changeset/113323 helps.

I unskipped the tests in http://trac.webkit.org/changeset/113419
But leave the bug open for 1-2 days to make sure if these tests don't crash anymore.
Comment 12 Csaba Osztrogonác 2012-05-14 03:50:55 PDT
They don't crash long time ago, so we can close it now.