This is the same optimization made for ThreadSafeAutoRefCnt in bug 1277709. However, nsStringBuffer uses a 32-bit reference count all the time, so it can't easily use ThreadSafeAutoRefCnt. MozReview-Commit-ID: LpB3xaYbaEE

This is the same optimization made for ThreadSafeAutoRefCnt in bug 1277709. However, nsStringBuffer uses a 32-bit reference count all the time, so it can't easily use ThreadSafeAutoRefCnt. MozReview-Commit-ID: LpB3xaYbaEE

Comment on attachment 8854185 [details] [diff] [review] Make nsStringBuffer's reference count use the minimum memory synchronization needed Review of attachment 8854185 [details] [diff] [review]: ----------------------------------------------------------------- Thanks for the comment in IsReadonly().

https://treeherder.mozilla.org/#/jobs?repo=try&revision=50f3c56e5af5be0e01c02a67496ae5da91f7a8ac&group_state=expanded looks pretty green (though not *quite* done yet).

https://hg.mozilla.org/integration/mozilla-inbound/rev/b6f0b34e87f26826fb65c9889d6d1f24fc6f200c Bug 1353181 - Make nsStringBuffer's reference count use the minimum memory synchronization needed. r=froydnj

https://hg.mozilla.org/integration/mozilla-inbound/rev/19c4e80bee84eb2515f71df3649423245d0c13d5 Bug 1353181 followup - Fix non-DEBUG warnings-as-errors bustage.

https://hg.mozilla.org/mozilla-central/rev/b6f0b34e87f2 https://hg.mozilla.org/mozilla-central/rev/19c4e80bee84