[webkit-reviews] review cancelled: [Bug 22524] XMLHttpRequest needs to have the static defines made threadsafe. : [Attachment 25555] Very minor adjustment to the last patch (-- moves one brace for style purposes).

[bugzilla-daemon at webkit.org]

David Levin <levin at chromium.org> has cancelled David Levin
<levin at chromium.org>'s request for review:
Bug 22524: XMLHttpRequest needs to have the static defines made threadsafe.
https://bugs.webkit.org/show_bug.cgi?id=22524

Attachment 25555: Very minor adjustment to the last patch (-- moves one brace
for style purposes).
https://bugs.webkit.org/attachment.cgi?id=25555&action=review

------- Additional Comments from David Levin <levin at chromium.org>
I didn't like doing a mutex in the functions due to one of them being called
inside of a loop.   As discussed, I've tried out a solution that initializes
the static variables in the constructor.  I looked at a few ways of doing this.
 The reason I put the variables in a struct and then initialized into a local
static was to take advantage of AtomicallyInitializedStatic.

If this doesn't seem that great, one alternative I can think of is to create
another macro like AtomicallyInitializedStatic (in threading.h) that allowed me
to call a function inside of this same mutex.