Add standalone drawText for GrTextContext.

Add standalone drawText for GrTextContext.

This unifies the interface between GrBitmapTextContext and
GrDistanceFieldTextContext so that they don't need special case code.
The future GrNVPRTextContext will also use this interface.

BUG=skia:2018
Committed: http://code.google.com/p/skia/source/detail?r=13227

[jvanverth1, 2014-01-27 21:25:58]

I've verified that this doesn't affect the GMs, and am waiting on a Perf
testbot. The code in BitmapTextContext copies a lot of what was going on in
SkDraw -- the plan after this change is to streamline that, if possible.

[bsalomon, 2014-01-27 21:51:25]

https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrDistanceF...
File include/gpu/GrDistanceFieldTextContext.h (right):

https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrDistanceF...
include/gpu/GrDistanceFieldTextContext.h:20: void drawText(const char text[],
size_t byteLength, SkScalar x, SkScalar y) SK_OVERRIDE;
virtual? (and several more)

https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrTextConte...
File include/gpu/GrTextContext.h (right):

https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrTextConte...
include/gpu/GrTextContext.h:65: ManagedTextContext(SkGpuDevice* device,
Is this just to get at the device properties? I'd like to somehow encapsulate
just what is necessary. device has a lot of functions that I don't think we'd
want a text context subclass to call.

[jvanverth1, 2014-01-28 13:42:19]

On 2014/01/27 21:51:25, bsalomon wrote:
>
https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrDistanceF...
> File include/gpu/GrDistanceFieldTextContext.h (right):
> 
>
https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrDistanceF...
> include/gpu/GrDistanceFieldTextContext.h:20: void drawText(const char text[],
> size_t byteLength, SkScalar x, SkScalar y) SK_OVERRIDE;
> virtual? (and several more)
> 
>
https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrTextConte...
> File include/gpu/GrTextContext.h (right):
> 
>
https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrTextConte...
> include/gpu/GrTextContext.h:65: ManagedTextContext(SkGpuDevice* device,
> Is this just to get at the device properties? I'd like to somehow encapsulate
> just what is necessary. device has a lot of functions that I don't think we'd
> want a text context subclass to call.

Ok, I can store the device properties instead.

[jvanverth1, 2014-01-28 18:05:33]

https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrDistanceF...
File include/gpu/GrDistanceFieldTextContext.h (right):

https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrDistanceF...
include/gpu/GrDistanceFieldTextContext.h:20: void drawText(const char text[],
size_t byteLength, SkScalar x, SkScalar y) SK_OVERRIDE;
On 2014/01/27 21:51:25, bsalomon wrote:
> virtual? (and several more)

Done.

https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrTextConte...
File include/gpu/GrTextContext.h (right):

https://codereview.chromium.org/141863005/diff/200001/include/gpu/GrTextConte...
include/gpu/GrTextContext.h:65: ManagedTextContext(SkGpuDevice* device,
On 2014/01/27 21:51:25, bsalomon wrote:
> Is this just to get at the device properties? I'd like to somehow encapsulate
> just what is necessary. device has a lot of functions that I don't think we'd
> want a text context subclass to call.

Done.

[bsalomon, 2014-01-28 18:46:05]

lgtm

Keeping a ptr to the non-ref-counted device properties seems a little dubious.
Though, I get that currently it will never be dangling currently. AFAICT the
properties are a byte and a float so making a copy doesn't seem expensive. I
leave it up to you.

https://codereview.chromium.org/141863005/diff/220001/src/gpu/GrDistanceField...
File src/gpu/GrDistanceFieldTextContext.cpp (right):

https://codereview.chromium.org/141863005/diff/220001/src/gpu/GrDistanceField...
src/gpu/GrDistanceFieldTextContext.cpp:302: SkGlyphCache*       cache =
autoCache.getCache();
whitespace?

https://codereview.chromium.org/141863005/diff/220001/src/gpu/GrDistanceField...
src/gpu/GrDistanceFieldTextContext.cpp:371: SkGlyphCache*       cache =
autoCache.getCache();
whitespace?

[commit-bot: I haz the power, 2014-01-28 20:08:42]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/jvanverth@google.com/141863005/340001

[commit-bot: I haz the power, 2014-01-28 20:29:59]

Retried try job too often on Build-Mac10.7-Clang-x86-Release-Trybot for step(s)
BuildSkiaLib
http://skiabot-master.pogerlabs.com:10117/buildstatus?builder=Build-Mac10.7-C...

[commit-bot: I haz the power, 2014-01-28 21:23:06]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/jvanverth@google.com/141863005/380001

[commit-bot: I haz the power, 2014-01-28 22:02:11]

Change committed as 13227

[commit-bot: I haz the power, 2014-01-28 22:03:54]

CQ bit was unchecked on CL. Ignoring.