Make all GrContext members that return a texture also ref the texture for the caller.

Make all GrContext members that return a texture also ref the texture for the caller.
Committed: https://code.google.com/p/skia/source/detail?r=7362

[bsalomon, 2013-01-24 14:30:32]

This makes the current code safer. It also will allow the cache to drop its ref,
enabling the client's unref() to trigger internal_dispose().

https://codereview.appspot.com/7198049/diff/3001/tests/ClipCacheTest.cpp
File tests/ClipCacheTest.cpp (right):

https://codereview.appspot.com/7198049/diff/3001/tests/ClipCacheTest.cpp#newc...
tests/ClipCacheTest.cpp:158: REPORTER_ASSERT(reporter, texture1->getRefCnt());
I didn't understand these asserts and am not sure if my change is correct. Now
there are two refs on these textures but it seems weird to expect a specific
count here.

[robertphillips, 2013-01-24 14:45:43]

LGTM

https://codereview.appspot.com/7198049/diff/3001/tests/ClipCacheTest.cpp
File tests/ClipCacheTest.cpp (right):

https://codereview.appspot.com/7198049/diff/3001/tests/ClipCacheTest.cpp#newc...
tests/ClipCacheTest.cpp:158: REPORTER_ASSERT(reporter, texture1->getRefCnt());
They are left over from an earlier vision of how the cache would work. I think
this test will have to be rewritten when I get back to it.

[robertphillips, 2013-01-24 14:46:42]

Does the instance count at exit on the gm's remain the same?

[bsalomon, 2013-01-24 14:47:56]

On 2013/01/24 14:46:42, robertphillips wrote:
> Does the instance count at exit on the gm's remain the same?

I that checked bench, tests, and gm don't leak GrResources according to the
instance counter.