Small change to move GrProcessor and GrBatch pools over to SkSpinlock

Small change to move GrProcessor and GrBatch pools over to SkSpinlock

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/23ac62c83a49d675a38f1c20462b5537f3c8af01

[joshualitt, 2015-03-30 15:18:33]

joshualitt@google.com changed reviewers:
+ bsalomon@google.com, mtklein@google.com

[joshualitt, 2015-03-30 15:20:11]

joshualitt@google.com changed reviewers:
+ joshualitt@google.com

[joshualitt, 2015-03-30 15:20:12]

The benefit here is pretty variable depending on the mutex cost per platform,
but its always atleast a small win.

[bsalomon, 2015-03-30 15:21:03]

https://codereview.chromium.org/1040133002/diff/1/src/gpu/GrProcessor.cpp
File src/gpu/GrProcessor.cpp (left):

https://codereview.chromium.org/1040133002/diff/1/src/gpu/GrProcessor.cpp#old...
src/gpu/GrProcessor.cpp:100: // We use a global pool protected by a mutex.
Chrome may use the same GrContext on different
update comments

[bsalomon, 2015-03-30 15:21:18]

otherwise lgtm

[mtklein, 2015-03-30 15:24:10]

https://codereview.chromium.org/1040133002/diff/1/src/gpu/GrBatch.cpp
File src/gpu/GrBatch.cpp (right):

https://codereview.chromium.org/1040133002/diff/1/src/gpu/GrBatch.cpp#newcode23
src/gpu/GrBatch.cpp:23: static SkSpinlock gBatchSpinlock;
Slightly nicer to use SK_DECLARE_STATIC_SPINLOCK(gBatchSpinlock);

Today the effect is the same, but this lets me fiddle with the SkSpinlock impl
without breaking you.

https://codereview.chromium.org/1040133002/diff/1/src/gpu/GrProcessor.cpp
File src/gpu/GrProcessor.cpp (left):

https://codereview.chromium.org/1040133002/diff/1/src/gpu/GrProcessor.cpp#old...
src/gpu/GrProcessor.cpp:100: // We use a global pool protected by a mutex.
Chrome may use the same GrContext on different
On 2015/03/30 15:21:03, bsalomon wrote:
> update comments

(SkSpinlock is a mutex.  It's just not an SkMutex.  I don't know if that makes
things more or less confusing.)

[joshualitt, 2015-03-30 16:48:11]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-03-30 16:48:11]

The patchset sent to the CQ was uploaded after l-g-t-m from bsalomon@google.com
Link to the patchset: https://codereview.chromium.org/1040133002/#ps20001
(title: "feedback inc")

[commit-bot: I haz the power, 2015-03-30 16:48:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1040133002/20001

[commit-bot: I haz the power, 2015-03-30 16:53:52]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/23ac62c83a49d675a38f1c20462b5537f3c8af01