Move glyph choosing to the find and place glyph code.

Move glyph choosing to the find and place glyph code.

This duplicates the functionality of the (private)
SkPaint::getDrawCacheProc method into
SkFindAndPlaceGlyph::LookupGlyph. Eventually LookupGlyph
should replace getDrawCacheProc, at which point it should be
removed. The remaining users are gpu and pdf.

Committed: https://skia.googlesource.com/skia/+/4c11b3f8a2d9919a21110dbdd29e67e5cbaa41fb

[herb_g, 2015-11-20 17:48:10]

Comments and fixes.

[herb_g, 2015-11-20 18:04:59]

Description was changed from

==========
Move glyph choosing to the find and place glyph code.

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

to

==========
Move glyph choosing to the find and place glyph code.

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

[herb_g, 2015-11-20 18:05:00]

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

[herb_g, 2015-11-20 18:05:58]

This CL will make it so I don't have to make XPS device a friend of SkPaint, and
possibly remove some friends in SkPaint.

[bungeman-skia, 2015-11-20 18:28:00]

Description was changed from

==========
Move glyph choosing to the find and place glyph code.

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

to

==========
Move glyph choosing to the find and place glyph code.

This duplicates the functionality of the (private)
SkPaint::getDrawCacheProc method into
SkFindAndPlaceGlyph::LookupGlyph. Eventually LookupGlyph
should replace getDrawCacheProc, at which point it should be
removed.
==========

[bungeman-skia, 2015-11-20 18:30:48]

Description was changed from

==========
Move glyph choosing to the find and place glyph code.

This duplicates the functionality of the (private)
SkPaint::getDrawCacheProc method into
SkFindAndPlaceGlyph::LookupGlyph. Eventually LookupGlyph
should replace getDrawCacheProc, at which point it should be
removed.
==========

to

==========
Move glyph choosing to the find and place glyph code.

This duplicates the functionality of the (private)
SkPaint::getDrawCacheProc method into
SkFindAndPlaceGlyph::LookupGlyph. Eventually LookupGlyph
should replace getDrawCacheProc, at which point it should be
removed. The remaining users are gpu and pdf.
==========

[bungeman-skia, 2015-11-20 18:30:59]

The CQ bit was checked by bungeman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2015-11-20 18:31:33]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1458193003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1458193003/20001

[bungeman-skia, 2015-11-20 19:04:11]

lgtm

[bungeman-skia, 2015-11-20 19:04:17]

The CQ bit was unchecked by bungeman@google.com

[bungeman-skia, 2015-11-20 19:04:22]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2015-11-20 19:04:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1458193003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1458193003/20001

[commit-bot: I haz the power, 2015-11-20 21:53:15]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/4c11b3f8a2d9919a21110dbdd29e67e5cbaa41fb