[Webkit-unassigned] [Bug 55402] loaderRunLoop() should use synchronization instead of while loop

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 18 19:57:58 PDT 2012


https://bugs.webkit.org/show_bug.cgi?id=55402





--- Comment #7 from Pratik Solanki <psolanki at apple.com>  2012-10-18 19:58:53 PST ---
(In reply to comment #5)
> (In reply to comment #3)
> Thanks for the review.
> 
> > (From update of attachment 169508 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=169508&action=review
> > 
> > Thanks for working on this bug. I don't think its quite right though. Have you tested on a platform that uses this file i.e. Windows?
> No, I haven't. But is there a difference in using threading primitives according to the platforms?

I only ask because this file is not compiled on all platforms. It's only compiled on Windows and iOS. So I would have more confidence in the patch if the code compiled and ran on Windows.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list