CLOSED FIXED 37316
[Qt] tests/hybridPixmap fails 
https://bugs.webkit.org/show_bug.cgi?id=37316
Summary [Qt] tests/hybridPixmap fails
Simon Hausmann
Reported 2010-04-09 02:47:03 PDT
WebKit/qt/tests/hybridPixmap fails on Linux (most likely on all platforms): ********* Start testing of tst_hybridPixmap ********* Config: Using QTest library 4.6.3, Qt 4.6.3 PASS : tst_hybridPixmap::initTestCase() ASSERTION FAILED: m_runtimeObject (/home/shausman/src/webkit/trunk/WebCore/bridge/jsc/BridgeJSC.cpp:108 void JSC::Bindings::Instance::willDestroyRuntimeObject()) QFATAL : tst_hybridPixmap::hybridPixmap() Received signal 11 FAIL! : tst_hybridPixmap::hybridPixmap() Received a fatal error. Loc: [Unknown file(0)] Totals: 1 passed, 1 failed, 0 skipped ********* Finished testing of tst_hybridPixmap ********* zsh: abort ./hybridPixmap
Attachments
Patch (4.77 KB, patch)
2010-06-12 18:54 PDT, Noam Rosenthal 		no flags
DetailsFormatted DiffDiff
Patch (3.83 KB, patch)
2010-06-12 18:58 PDT, Noam Rosenthal 		no flags
DetailsFormatted DiffDiff
Patch (3.78 KB, patch)
2010-06-12 19:10 PDT, Noam Rosenthal 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Simon Hausmann
Comment 1 2010-04-26 06:42:00 PDT
Noam, can you look into this crash? Thanks :)
Noam Rosenthal
Comment 2 2010-04-26 18:32:09 PDT
will have a look!
Diego Gonzalez
Comment 3 2010-05-04 11:50:25 PDT
I did no get this test failing anymore. Does anyone is getting this test still failing?
Jesus Sanchez-Palencia
Comment 4 2010-05-04 13:35:50 PDT
(In reply to comment #3) > I did no get this test failing anymore. Does anyone is getting this test still > failing? Tested on Snow Leopard with Qt 4.7 and it is not failing anymore. Diego you did test on Linux and on Maemo, right? Maybe this was fixed and the bug can be closed...
Noam Rosenthal
Comment 5 2010-05-04 15:19:03 PDT
I never got this to reproduce.
Simon Hausmann
Comment 6 2010-05-05 03:09:27 PDT
Still happens for me with ToT r58808 on Linux. Note: The assertion only fails in debug builds!
Noam Rosenthal
Comment 7 2010-05-05 03:12:04 PDT
That explains why it wasn't reproduced - I'll get a debug build running :)
Jesus Sanchez-Palencia
Comment 8 2010-05-05 08:11:15 PDT
(In reply to comment #7) > That explains why it wasn't reproduced - I'll get a debug build running :) Tested again with QtWebKit r58755 and Qt 4.7 HEAD 32ecf8e8ad326ea13ec9a430c99ce540e8b4efac in a debug build and got no fails on Mac OS X (Snow Leopard).
Diego Gonzalez
Comment 9 2010-05-05 10:34:17 PDT
(In reply to comment #8) > (In reply to comment #7) > > That explains why it wasn't reproduced - I'll get a debug build running :) > > Tested again with QtWebKit r58755 and Qt 4.7 HEAD > 32ecf8e8ad326ea13ec9a430c99ce540e8b4efac > in a debug build and got no fails on Mac OS X (Snow Leopard). The same happens with Linux in debug mode
Jesus Sanchez-Palencia
Comment 10 2010-05-06 13:43:55 PDT
*** Bug 38681 has been marked as a duplicate of this bug. ***
Simon Hausmann
Comment 11 2010-05-14 09:06:29 PDT
P1 -> P2. Good for patch release, but not blocking ATM.
Kenneth Rohde Christiansen
Comment 12 2010-06-11 18:59:42 PDT
no'am are you still looking into this? It would be really nice if you could have a look at it.
Noam Rosenthal
Comment 13 2010-06-11 19:19:27 PDT
Got jammed with other stuff. WIll look when I get a chance :)
Noam Rosenthal
Comment 14 2010-06-12 18:54:58 PDT
Created attachment 58570 [details] Patch
Noam Rosenthal
Comment 15 2010-06-12 18:58:37 PDT
Created attachment 58571 [details] Patch
Noam Rosenthal
Comment 16 2010-06-12 19:10:24 PDT
Created attachment 58572 [details] Patch
WebKit Commit Bot
Comment 17 2010-06-13 06:05:20 PDT
Comment on attachment 58572 [details] Patch Clearing flags on attachment: 58572 Committed r61089: <http://trac.webkit.org/changeset/61089>
WebKit Commit Bot
Comment 18 2010-06-13 06:05:27 PDT
All reviewed patches have been landed. Closing bug.
Simon Hausmann
Comment 19 2010-06-13 14:03:15 PDT
Revision r61089 cherry-picked into qtwebkit-2.0 with commit c4d3ca566aa29282263d7fdf4396274de715a489
Note You need to log in before you can comment on or make changes to this bug.