Bug 33581

Summary:

websocket/tests/worker/worker-simple.html crash on Leopard Intel Debug (Tests) builder

Product:

WebKit

Reporter:

Fumitoshi Ukai <ukai>

Component:

WebCore Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

levin

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
Patch	levin: review+

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>