Bug 37316 - [Qt] tests/hybridPixmap fails
Summary: [Qt] tests/hybridPixmap fails
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All Linux
: P2 Blocker
Assignee: Noam Rosenthal
URL:
Keywords: Qt, QtTriaged
: 38681 (view as bug list)
Depends on:
Blocks: 38654
  Show dependency tree / graph
 
Reported: 2010-04-09 02:47 PDT by Simon Hausmann
Modified: 2010-06-13 14:03 PDT (History)
4 users (show)

See Also:

Attachments
Patch (4.77 KB, patch)
2010-06-12 18:54 PDT, Noam Rosenthal 		no flags Details | Formatted Diff | Diff
Patch (3.83 KB, patch)
2010-06-12 18:58 PDT, Noam Rosenthal 		no flags Details | Formatted Diff | Diff
Patch (3.78 KB, patch)
2010-06-12 19:10 PDT, Noam Rosenthal 		no flags Details | Formatted Diff | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Hausmann 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
Comment 1 Simon Hausmann 2010-04-26 06:42:00 PDT 
Noam, can you look into this crash? Thanks :)
Comment 2 Noam Rosenthal 2010-04-26 18:32:09 PDT 
will have a look!
Comment 3 Diego Gonzalez 2010-05-04 11:50:25 PDT 
I did no get this test failing anymore. Does anyone is getting this test still failing?
Comment 4 Jesus Sanchez-Palencia 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...
Comment 5 Noam Rosenthal 2010-05-04 15:19:03 PDT 
I never got this to reproduce.
Comment 6 Simon Hausmann 2010-05-05 03:09:27 PDT 
Still happens for me with ToT r58808 on Linux.

Note: The assertion only fails in debug builds!
Comment 7 Noam Rosenthal 2010-05-05 03:12:04 PDT 
That explains why it wasn't reproduced - I'll get a debug build running :)
Comment 8 Jesus Sanchez-Palencia 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).
Comment 9 Diego Gonzalez 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
Comment 10 Jesus Sanchez-Palencia 2010-05-06 13:43:55 PDT 
*** Bug 38681 has been marked as a duplicate of this bug. ***
Comment 11 Simon Hausmann 2010-05-14 09:06:29 PDT 
P1 -> P2. Good for patch release, but not blocking ATM.
Comment 12 Kenneth Rohde Christiansen 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.
Comment 13 Noam Rosenthal 2010-06-11 19:19:27 PDT 
Got jammed with other stuff. WIll look when I get a chance :)
Comment 14 Noam Rosenthal 2010-06-12 18:54:58 PDT 
Created attachment 58570 [details]
Patch
Comment 15 Noam Rosenthal 2010-06-12 18:58:37 PDT 
Created attachment 58571 [details]
Patch
Comment 16 Noam Rosenthal 2010-06-12 19:10:24 PDT 
Created attachment 58572 [details]
Patch
Comment 17 WebKit Commit Bot 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>
Comment 18 WebKit Commit Bot 2010-06-13 06:05:27 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 19 Simon Hausmann 2010-06-13 14:03:15 PDT 
Revision r61089 cherry-picked into qtwebkit-2.0 with commit c4d3ca566aa29282263d7fdf4396274de715a489