Bug 106016 - [Qt] New fast/workers/worker-document-leak.html fails
Summary: [Qt] New fast/workers/worker-document-leak.html fails
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 87008 105367
  Show dependency treegraph
 
Reported: 2013-01-03 06:35 PST by Csaba Osztrogonác
Modified: 2014-02-03 03:24 PST (History)
4 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 2013-01-03 06:35:25 PST
fast/workers/worker-document-leak.html introduced in https://trac.webkit.org/changeset/138693,
but fails on Qt with the following diff:

--- /ramdisk/qt-linux-64-release/build/layout-test-results/fast/workers/worker-document-leak-expected.txt 
+++ /ramdisk/qt-linux-64-release/build/layout-test-results/fast/workers/worker-document-leak-actual.txt 
@@ -1,4 +1,4 @@
 Verify that creation of a worker does not leak its creating document.
 
-PASS: did not leak documents during test run
+FAIL: leaked documents during test run (started with 1, ended with 3)
Comment 1 Csaba Osztrogonác 2013-01-03 06:38:58 PST
I skipped it to paint the bots green: r138710. Please unskip it with the proper fix.
Comment 2 Kenneth Russell 2013-01-03 12:18:52 PST
I don't know how to build the Qt port on any platform, so unfortunately can't help debug what's going on. You could try adding more calls to gc() at the end of the test, to try to force collection of the objects associated with the worker and the iframe's DOM.

Qt uses JSC, correct? Since it does, I'd expect it to behave similarly to the Mac port with respect to garbage collection of JavaScript wrappers. The new test seems to be passing on all the Mac bots.
Comment 3 Jocelyn Turcotte 2014-02-03 03:24:11 PST
=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.