Remove pthread dependency of GCController
Created attachment 83102 [details] Patch
The patch looks fine to me, but the code as a whole doesn't. Shouldn't we call pthread_detach() in immediate return case? CC'ing the author of this code in case I'm somehow confused.
(In reply to comment #2) > Shouldn't we call pthread_detach() in immediate return case? Yes. This is a bug.
Comment on attachment 83102 [details] Patch (In reply to comment #3) > (In reply to comment #2) > > Shouldn't we call pthread_detach() in immediate return case? > > Yes. This is a bug. I'll create a new patch in the next days.
Created attachment 83363 [details] Patch
Is it safe to call detachThread after calling waitForThreadCompletion?
I thought that it's one or the other, not both.
Created attachment 83366 [details] Patch AFAIK it's no problem to do both, but the cleaner way is to to only one of them
Comment on attachment 83366 [details] Patch Seems sane to me.
Comment on attachment 83366 [details] Patch Clearing flags on attachment: 83366 Committed r84113: <http://trac.webkit.org/changeset/84113>
All reviewed patches have been landed. Closing bug.
The commit-queue encountered the following flaky tests while processing attachment 83366 [details]: animations/suspend-resume-animation.html bug 48161 (author: cmarrin@apple.com) The commit-queue is continuing to process your patch.
The commit-queue encountered the following flaky tests while processing attachment 83366 [details]: http/tests/xmlhttprequest/abort-crash.html bug 51649 (author: andersca@apple.com) The commit-queue is continuing to process your patch.