Use sk_sp text blob APIs

Use sk_sp text blob APIs

SkTextBlobBuilder::build()      -> make()
SkAutoTUnref<const SkTextBlob>  -> sk_sp<SkTextBlob>
drawTextBlob(const SkTextBlob*) -> drawTextBlob(const sk_sp<SkTextBlob>&)


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

Committed: https://skia.googlesource.com/skia/+/37283c28aa5bea2204c18956e74f83b238d7a891

[f(malita), 2016-09-12 17:20:28]

Description was changed from

==========
Use sk_sp text blob APIs

SkTextBlobBuilder::build()      -> make()
SkAutoTUnref<const SkTextBlob>  -> sk_sp<SkTextBlob>
drawTextBlob(const SkTextBlob*) -> drawTextBlob(const sk_sp<SkTextBlob>&)


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

to

==========
Use sk_sp text blob APIs

SkTextBlobBuilder::build()      -> make()
SkAutoTUnref<const SkTextBlob>  -> sk_sp<SkTextBlob>
drawTextBlob(const SkTextBlob*) -> drawTextBlob(const sk_sp<SkTextBlob>&)


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

[f(malita), 2016-09-12 17:23:10]

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

[f(malita), 2016-09-12 17:23:11]

Depends on https://codereview.chromium.org/2335703002/

We may need to leave the old style SkTextBlobBuilder::build() alone for now if
used in Android/G3.

[f(malita), 2016-09-12 17:23:16]

The CQ bit was checked by fmalita@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-12 17:23:29]

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

[commit-bot: I haz the power, 2016-09-12 18:11:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-12 18:11:45]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[robertphillips, 2016-09-13 12:38:00]

lgtm

https://codereview.chromium.org/2335493005/diff/1/src/gpu/text/GrStencilAndCo...
File src/gpu/text/GrStencilAndCoverTextContext.cpp (right):

https://codereview.chromium.org/2335493005/diff/1/src/gpu/text/GrStencilAndCo...
src/gpu/text/GrStencilAndCoverTextContext.cpp:346: 
makeIfNeeded ?

[f(malita), 2016-09-13 13:41:57]

https://codereview.chromium.org/2335493005/diff/1/src/gpu/text/GrStencilAndCo...
File src/gpu/text/GrStencilAndCoverTextContext.cpp (right):

https://codereview.chromium.org/2335493005/diff/1/src/gpu/text/GrStencilAndCo...
src/gpu/text/GrStencilAndCoverTextContext.cpp:346: 
On 2016/09/13 12:37:59, robertphillips wrote:
> makeIfNeeded ?

Done.

[f(malita), 2016-09-13 13:42:04]

The CQ bit was checked by fmalita@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-13 13:42:09]

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

[commit-bot: I haz the power, 2016-09-13 14:23:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 14:23:23]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[f(malita), 2016-09-13 16:32:38]

Chrome & Android are now clear, but I G3 is likely still dependent on the old
builder API.

Added guard + bzl define in PS#4.

I think this needs Mike's public API blessing.

[f(malita), 2016-09-13 16:32:43]

The CQ bit was checked by fmalita@chromium.org

[f(malita), 2016-09-13 16:32:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/2335493005/#ps40001
(title: "SK_SUPPORT_LEGACY_TEXTBLOB_BUILDER")

[commit-bot: I haz the power, 2016-09-13 16:32:50]

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

[commit-bot: I haz the power, 2016-09-13 16:34:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-13 16:34:56]

Try jobs failed on following builders:
  skia_presubmit-Trybot on master.client.skia.fyi (JOB_FAILED,
http://build.chromium.org/p/client.skia.fyi/builders/skia_presubmit-Trybot/bu...)

[reed1, 2016-09-13 16:58:44]

lgtm

[f(malita), 2016-09-13 16:59:22]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2016-09-13 16:59:26]

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

[commit-bot: I haz the power, 2016-09-13 17:00:25]

Description was changed from

==========
Use sk_sp text blob APIs

SkTextBlobBuilder::build()      -> make()
SkAutoTUnref<const SkTextBlob>  -> sk_sp<SkTextBlob>
drawTextBlob(const SkTextBlob*) -> drawTextBlob(const sk_sp<SkTextBlob>&)


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

to

==========
Use sk_sp text blob APIs

SkTextBlobBuilder::build()      -> make()
SkAutoTUnref<const SkTextBlob>  -> sk_sp<SkTextBlob>
drawTextBlob(const SkTextBlob*) -> drawTextBlob(const sk_sp<SkTextBlob>&)


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

Committed:
https://skia.googlesource.com/skia/+/37283c28aa5bea2204c18956e74f83b238d7a891
==========

[commit-bot: I haz the power, 2016-09-13 17:00:26]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/37283c28aa5bea2204c18956e74f83b238d7a891