Bug 83749

Summary: Leak in WebSocketChannel with workers/worker-reload.html (part 2)
Product: WebKit Reporter: Yuta Kitamura <yutak>
Component: WebCore Misc.Assignee: Yuta Kitamura <yutak>
Status: RESOLVED FIXED    
Severity: Normal CC: levin, levin+threading, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Description Yuta Kitamura 2012-04-11 23:30:05 PDT 
Follow-up of bug 83345. See also http://code.google.com/p/chromium/issues/detail?id=122233

Even after a speculative fix at r113414, there are still some similar leaks reported. After some investigation I found:
- A silly mistake in Chromium's WebWorkerClientImpl, and
- Potential race condition that causes Peer objects to leak.
Comment 1 Yuta Kitamura 2012-04-12 03:02:49 PDT 
Created attachment 136863 [details]
Patch
Comment 2 WebKit Review Bot 2012-04-15 18:43:56 PDT 
Comment on attachment 136863 [details]
Patch

Clearing flags on attachment: 136863

Committed r114215: <http://trac.webkit.org/changeset/114215>
Comment 3 WebKit Review Bot 2012-04-15 18:44:00 PDT 
All reviewed patches have been landed.  Closing bug.