GrBufferAllocPool: use _throw calloc() if unchecked.

GrBufferAllocPool: use _throw calloc() if unchecked.

Looks like this code is using sk_calloc(), NULL on failure, accidentally
instead of sk_calloc_throw().  We're using sk_malloc_throw() in the parallel
code path, so it really seems like we're not checking the result pointer.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2152753002

Committed: https://skia.googlesource.com/skia/+/abda35d2b77f2765985ab16d3417c5cce296b8af

[mtklein_C, 2016-07-14 13:39:22]

Description was changed from

==========
GrBufferAllocPool: use _throw calloc() if unchecked.

Looks like this code is using sk_calloc(), NULL on failure, accidentally
instead of sk_calloc_throw().  We're using sk_malloc_throw() in the parallel
code path, so it really seems like we're not checking the result pointer.

BUG=skia:
==========

to

==========
GrBufferAllocPool: use _throw calloc() if unchecked.

Looks like this code is using sk_calloc(), NULL on failure, accidentally
instead of sk_calloc_throw().  We're using sk_malloc_throw() in the parallel
code path, so it really seems like we're not checking the result pointer.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2152753002
==========

[mtklein_C, 2016-07-14 13:40:23]

mtklein@chromium.org changed reviewers:
+ bsalomon@google.com

[mtklein_C, 2016-07-14 13:40:23]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2016-07-14 13:40:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bsalomon, 2016-07-14 13:47:55]

lgtm

[commit-bot: I haz the power, 2016-07-14 13:57:33]

Description was changed from

==========
GrBufferAllocPool: use _throw calloc() if unchecked.

Looks like this code is using sk_calloc(), NULL on failure, accidentally
instead of sk_calloc_throw().  We're using sk_malloc_throw() in the parallel
code path, so it really seems like we're not checking the result pointer.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2152753002
==========

to

==========
GrBufferAllocPool: use _throw calloc() if unchecked.

Looks like this code is using sk_calloc(), NULL on failure, accidentally
instead of sk_calloc_throw().  We're using sk_malloc_throw() in the parallel
code path, so it really seems like we're not checking the result pointer.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2152753002

Committed:
https://skia.googlesource.com/skia/+/abda35d2b77f2765985ab16d3417c5cce296b8af
==========

[commit-bot: I haz the power, 2016-07-14 13:57:35]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/abda35d2b77f2765985ab16d3417c5cce296b8af