superreview requested: [Bug 720778] Important threads should have a name for better debugability : [Attachment 627927] v3.1

[bugzill...@mozilla.org]

Honza Bambas (:mayhemer) <honza...@firemni.cz> has asked Benjamin Smedberg
[:bsmedberg] <benj...@smedbergs.us> for superreview:
Bug 720778: Important threads should have a name for better debugability
https://bugzilla.mozilla.org/show_bug.cgi?id=720778

Attachment 627927: v3.1
https://bugzilla.mozilla.org/attachment.cgi?id=627927&action=edit


------- Additional Comments from Honza Bambas (:mayhemer)
<honza...@firemni.cz>
- bsmith's last review comments addressed
- changed use of PR_AtomicIncrement to just volatile ++
- asking for SR

[bugzill...@mozilla.org]

Benjamin Smedberg [:bsmedberg] <benj...@smedbergs.us> has granted Honza
Bambas (:mayhemer) <honza...@firemni.cz>'s request for superreview:

Bug 720778: Important threads should have a name for better debugability
https://bugzilla.mozilla.org/show_bug.cgi?id=720778

Attachment 627927: v3.1
https://bugzilla.mozilla.org/attachment.cgi?id=627927&action=edit

------- Additional Comments from Benjamin Smedberg [:bsmedberg]
<benj...@smedbergs.us>
glandium, are these static classes (nsThreadPoolNaming) a problem for static
constructor counts?

[bugzill...@mozilla.org]

Honza Bambas (:mayhemer) <honza...@firemni.cz> has granted superreview:

Bug 720778: Important threads should have a name for better debugability
https://bugzilla.mozilla.org/show_bug.cgi?id=720778

Attachment 632274: v3.1 unbitrotted
https://bugzilla.mozilla.org/attachment.cgi?id=632274&action=edit