RESOLVED INVALID 98640
[Qt] plugins/refcount-leaks.html fails 
https://bugs.webkit.org/show_bug.cgi?id=98640
Summary [Qt] plugins/refcount-leaks.html fails
Csaba Osztrogonác
Reported 2012-10-08 03:56:43 PDT
Plugin tests were unskipped by https://trac.webkit.org/changeset/130623, but this one still fail. WK1 fail: ---------- --- /home/oszi/WebKit/WebKitBuild/Release/layout-test-results/plugins/refcount-leaks-expected.txt +++ /home/oszi/WebKit/WebKitBuild/Release/layout-test-results/plugins/refcount-leaks-actual.txt @@ -8,5 +8,5 @@ --- refcount on plug.testObject: originally: 2 after GC: 2 -after passing: 2 -SUCCESS +after passing: 3 +FAILURE WK2 fail: ---------- --- /home/oszi/WebKit/WebKitBuild/Release/layout-test-results/plugins/refcount-leaks-expected.txt +++ /home/oszi/WebKit/WebKitBuild/Release/layout-test-results/plugins/refcount-leaks-actual.txt @@ -3,10 +3,10 @@ --- num test objects: countAfterCreate == countOrig + 3? PASS -countOrig == countAfterGC? PASS +countOrig == countAfterGC? FAIL --- refcount on plug.testObject: originally: 2 -after GC: 2 -after passing: 2 -SUCCESS +after GC: 5 +after passing: 8 +FAILURE
Attachments
Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2012-10-08 04:04:58 PDT
Skipped by r130632, please unskip it with the proper fix.
Csaba Osztrogonác
Comment 2 2012-11-21 04:50:34 PST
It passes on Qt5-WK1, but still fails on Qt5-WK2, so I moved the skip list entry to the WK2 platform - r135386.
Csaba Osztrogonác
Comment 3 2012-11-21 05:34:00 PST
I meant r135389.
Csaba Osztrogonác
Comment 4 2012-11-22 08:22:12 PST
It still fails on 32 bit Qt5-WK1 platform. (but passes on 64 bit)
Csaba Osztrogonác
Comment 5 2012-11-22 08:26:44 PST
I skipped it again - r135526.
Allan Sandfeld Jensen
Comment 6 2012-12-04 09:41:03 PST
It fails here on WK1-64bit as well. Inserting timeouts into the tests seems to help, so apparently there is a race condition of sorts. On WK2 though I get " line 24: TypeError: 'undefined' is not a function (evaluating 'plug.testCreateTestObject()" though. So it is even support in WebKitTestRunner?
Note You need to log in before you can comment on or make changes to this bug.