gDEBUG_LiveProtoCount is a debug-only counter of the number of live XPCWrappedNativeProtos. Currently, it is updated with PR_ATOMIC_INCREMENT/DECREMENT, presumably because XPConnect used to be multithreaded, but this is now unnecessary. This code looks essentially unchanged since the initial XPCDOM landing.
Created attachment 8775205 [details] [diff] [review] gDEBUG_LiveProtoCount does not need to be modified atomically. try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=de6442645240
Attachment #8775205 - Flags: review?(mrbkap)
Attachment #8775205 - Flags: review?(mrbkap) → review+
Thanks for all of the various reviews.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/de70d037b6b9 gDEBUG_LiveProtoCount does not need to be modified atomically. r=mrbkap
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox50: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.