1202667 - micro-optimize refcounting in TaskQueue

Status: RESOLVED FIXED

Description

[Nathan Froyd [:froydnj]]

This is micro-optimizing, but it feels worthwhile, because the refcounting here
on runnables is thread-safe, and avoiding atomic operations seems like a good
idea.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: make TaskQueue dispatching slightly more efficient

We're already holding a reference to the Runner prior to dispatching it
to the thread pool; we can pass that reference in rather than requiring
the thread pool to take a new reference to it.

Comment 2

[Nathan Froyd [:froydnj]]

Attachment: make TaskQueue task running slightly more efficient

Comment 3

[Andrew McCreight [:mccr8]]

I CC'ed Olli because he likes refcounting microoptimizations. ;)

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/348b09b8d643
https://hg.mozilla.org/integration/mozilla-inbound/rev/fe19d71aae51

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/348b09b8d643
https://hg.mozilla.org/mozilla-central/rev/fe19d71aae51