Initial change to move text blob to GrTextContext

Initial change to move text blob to GrTextContext

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/9c328187d9dee33736b77dc14dfb59529d948bb1

[joshualitt, 2015-03-18 19:59:45]

joshualitt@google.com changed reviewers:
+ joshualitt@google.com, jvanverth@google.com, robertphillips@google.com

[joshualitt, 2015-03-18 19:59:46]

The logic mostly comes from SkDevice.cpp

[robertphillips, 2015-03-18 20:05:12]

looks good to me but I would again like Jim to chime in

https://codereview.chromium.org/1019633002/diff/1/src/gpu/GrTextContext.h
File src/gpu/GrTextContext.h (right):

https://codereview.chromium.org/1019633002/diff/1/src/gpu/GrTextContext.h#new...
src/gpu/GrTextContext.h:41: void drawTextBlob(GrRenderTarget*, const GrClip&,
const SkPaint&,
It is probably relevant that this is the viewMatrix

[joshualitt, 2015-03-19 17:06:36]

On 2015/03/18 20:05:12, robertphillips wrote:
> looks good to me but I would again like Jim to chime in
> 
> https://codereview.chromium.org/1019633002/diff/1/src/gpu/GrTextContext.h
> File src/gpu/GrTextContext.h (right):
> 
>
https://codereview.chromium.org/1019633002/diff/1/src/gpu/GrTextContext.h#new...
> src/gpu/GrTextContext.h:41: void drawTextBlob(GrRenderTarget*, const GrClip&,
> const SkPaint&,
> It is probably relevant that this is the viewMatrix

okay I have updated this CL to be a bit more flexible.  It now respects the
textcontexts can draw, and provides a default implementation of onDrawTextBlob.

[jvanverth1, 2015-03-20 17:19:52]

On 2015/03/19 17:06:36, joshualitt wrote:
> On 2015/03/18 20:05:12, robertphillips wrote:
> > looks good to me but I would again like Jim to chime in
> > 
> > https://codereview.chromium.org/1019633002/diff/1/src/gpu/GrTextContext.h
> > File src/gpu/GrTextContext.h (right):
> > 
> >
>
https://codereview.chromium.org/1019633002/diff/1/src/gpu/GrTextContext.h#new...
> > src/gpu/GrTextContext.h:41: void drawTextBlob(GrRenderTarget*, const
GrClip&,
> > const SkPaint&,
> > It is probably relevant that this is the viewMatrix
> 
> okay I have updated this CL to be a bit more flexible.  It now respects the
> textcontexts can draw, and provides a default implementation of
onDrawTextBlob.

LGTM

[joshualitt, 2015-03-20 17:24:19]

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

[joshualitt, 2015-03-20 17:24:20]

Adding Reed for public api lgtm

[joshualitt, 2015-03-23 15:00:55]

On 2015/03/20 17:24:20, joshualitt wrote:
> Adding Reed for public api lgtm

Reed, can you ptal?

[reed1, 2015-03-23 15:04:48]

reed@google.com changed reviewers:
+ fmalita@chromium.org

[reed1, 2015-03-23 15:04:49]

lgtm

[joshualitt, 2015-03-23 15:07:37]

The CQ bit was checked by joshualitt@google.com

[commit-bot: I haz the power, 2015-03-23 15:07:47]

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

[commit-bot: I haz the power, 2015-03-23 15:13:08]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/9c328187d9dee33736b77dc14dfb59529d948bb1