Small change to add a callto retrieve a malloc SkDescriptor from SkPaint

Call to retrieve SkDescriptor for SkScalarContext from SkPaint.

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

[joshualitt, 2015-03-13 14:22:11]

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

[joshualitt, 2015-03-13 14:22:11]

PTAL

[reed1, 2015-03-13 14:26:45]

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

[reed1, 2015-03-13 14:26:46]

+ben

I'd rather not make this new api public. Can we friend the one caller, or place
this in a private header?

[joshualitt, 2015-03-13 14:42:58]

On 2015/03/13 14:26:46, reed1 wrote:
> +ben
> 
> I'd rather not make this new api public. Can we friend the one caller, or
place
> this in a private header?

I think this is in the private section of SkPaint.

Updated the CL to make this function const and to return a const SkData instead
of just an SkDescriptor

[bungeman-skia, 2015-03-13 14:48:30]

https://codereview.chromium.org/1005103003/diff/20001/include/core/SkPaint.h
File include/core/SkPaint.h (right):

https://codereview.chromium.org/1005103003/diff/20001/include/core/SkPaint.h#...
include/core/SkPaint.h:1088: const SkData* getDescriptor(const
SkDeviceProperties* deviceProperties, const SkMatrix*,
This name if a bit misleading. It doesn't return a descriptor for the paint, but
a descriptor for a scaler context. The comment doesn't even specify this.

[joshualitt, 2015-03-13 14:56:26]

On 2015/03/13 14:48:30, bungeman1 wrote:
> https://codereview.chromium.org/1005103003/diff/20001/include/core/SkPaint.h
> File include/core/SkPaint.h (right):
> 
>
https://codereview.chromium.org/1005103003/diff/20001/include/core/SkPaint.h#...
> include/core/SkPaint.h:1088: const SkData* getDescriptor(const
> SkDeviceProperties* deviceProperties, const SkMatrix*,
> This name if a bit misleading. It doesn't return a descriptor for the paint,
but
> a descriptor for a scaler context. The comment doesn't even specify this.

Changed name to getScalerContextDescriptor

[reed1, 2015-03-13 15:15:18]

reed@google.com changed required reviewers:
+ bungeman@google.com

[reed1, 2015-03-13 15:15:19]

api lgtm

[bungeman-skia, 2015-03-13 15:28:27]

Why not? lgtm

[joshualitt, 2015-03-13 15:32:24]

The CQ bit was checked by joshualitt@google.com

[commit-bot: I haz the power, 2015-03-13 15:32:45]

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

[commit-bot: I haz the power, 2015-03-13 15:38:48]

Committed patchset #3 (id:20002) as
https://skia.googlesource.com/skia/+/fd45079314b739979492601fc7a242aff24c58a3