Bug 33581 - websocket/tests/worker/worker-simple.html crash on Leopard Intel Debug (Tests) builder
Summary: websocket/tests/worker/worker-simple.html crash on Leopard Intel Debug (Tests...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-13 00:36 PST by Fumitoshi Ukai
Modified: 2010-01-13 18:46 PST (History)
1 user (show)

See Also:


Attachments
Patch (1.79 KB, patch)
2010-01-13 00:40 PST, Fumitoshi Ukai
levin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fumitoshi Ukai 2010-01-13 00:36:23 PST
See detail for example:
http://build.webkit.org/results/Leopard%20Intel%20Debug%20(Tests)/r53175%20(9185)/websocket/tests/workers/worker-simple-stderr.txt

Looks workerContext ref counted error.
Comment 1 Fumitoshi Ukai 2010-01-13 00:40:51 PST
Created attachment 46432 [details]
Patch
Comment 2 Fumitoshi Ukai 2010-01-13 01:23:26 PST
Committed r53182: <http://trac.webkit.org/changeset/53182>
Comment 3 Fumitoshi Ukai 2010-01-13 01:24:31 PST
http://trac.webkit.org/changeset/53182 is just adding the crash test in Skipped.
Comment 4 David Levin 2010-01-13 11:05:12 PST
Comment on attachment 46432 [details]
Patch

The overall patch seems fine. It would be nice to explain just a little (in the changelog) why not doing this causes a crash.
Comment 5 Fumitoshi Ukai 2010-01-13 18:23:22 PST
Committed r53226: <http://trac.webkit.org/changeset/53226>
Comment 6 Fumitoshi Ukai 2010-01-13 18:46:53 PST
Committed r53227: <http://trac.webkit.org/changeset/53227>