[webkit-reviews] review requested: [Bug 28795] fast/workers/worker-lifecycle.html crashes intermittently on snowleopard : [Attachment 38769] proposed patch

[bugzilla-daemon at webkit.org]

Andrew Wilson <atwilson at chromium.org> has asked  for review:
Bug 28795: fast/workers/worker-lifecycle.html crashes intermittently on
snowleopard
https://bugs.webkit.org/show_bug.cgi?id=28795

Attachment 38769: proposed patch
https://bugs.webkit.org/attachment.cgi?id=38769&action=review

------- Additional Comments from Andrew Wilson <atwilson at chromium.org>
Per my earlier conversation with ap - we no longer invoke WorkerThread::stop()
when WorkerContext::close() is called. Instead, the WorkerMessagingProxy and
SharedWorkerProxy shutdown the thread themselves, to avoid race conditions with
the thread shutting down while the main thread is still accessing it.