Extract glyph find and position code.

Extract glyph find and position code in preparation to use it in XPS and GPU code.

BUG=skia:

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

[herb_g, 2015-11-09 19:52:40]

Description was changed from

==========
Extract glyph find and position code.

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

to

==========
Extract glyph find and position code in preparation to use it in XPS and GPU
code.

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

[herb_g, 2015-11-09 19:52:40]

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

[mtklein, 2015-11-09 19:58:15]

lgtm

https://codereview.chromium.org/1409123010/diff/20001/src/core/SkDraw.cpp
File src/core/SkDraw.cpp (right):

https://codereview.chromium.org/1409123010/diff/20001/src/core/SkDraw.cpp#new...
src/core/SkDraw.cpp:1778: scalarsPerPosition,
funky indent on these args (and below)?

[herb_g, 2015-11-09 20:04:30]

https://codereview.chromium.org/1409123010/diff/20001/src/core/SkDraw.cpp
File src/core/SkDraw.cpp (right):

https://codereview.chromium.org/1409123010/diff/20001/src/core/SkDraw.cpp#new...
src/core/SkDraw.cpp:1778: scalarsPerPosition,
On 2015/11/09 19:58:15, mtklein wrote:
> funky indent on these args (and below)?

Done.

[herb_g, 2015-11-09 20:04:33]

The CQ bit was checked by herb@google.com

[herb_g, 2015-11-09 20:04:33]

The patchset sent to the CQ was uploaded after l-g-t-m from mtklein@google.com
Link to the patchset: https://codereview.chromium.org/1409123010/#ps40001
(title: "fix bad param formatting")

[commit-bot: I haz the power, 2015-11-09 20:04:41]

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

[commit-bot: I haz the power, 2015-11-09 20:06:37]

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

[commit-bot: I haz the power, 2015-11-09 20:06:37]

Try jobs failed on following builders:
  Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[herb_g, 2015-11-09 20:28:59]

The CQ bit was checked by herb@google.com

[herb_g, 2015-11-09 20:29:00]

The patchset sent to the CQ was uploaded after l-g-t-m from mtklein@google.com
Link to the patchset: https://codereview.chromium.org/1409123010/#ps60001
(title: "remove constexpr")

[commit-bot: I haz the power, 2015-11-09 20:29:09]

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

[herb_g, 2015-11-09 20:45:27]

The CQ bit was unchecked by herb@google.com

[herb_g, 2015-11-09 21:03:37]

The CQ bit was checked by herb@google.com

[herb_g, 2015-11-09 21:03:38]

The patchset sent to the CQ was uploaded after l-g-t-m from mtklein@google.com
Link to the patchset: https://codereview.chromium.org/1409123010/#ps80001
(title: "undef the constant")

[commit-bot: I haz the power, 2015-11-09 21:03:47]

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

[commit-bot: I haz the power, 2015-11-09 21:15:24]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/e5911c9c5b859d60208f4b9ac4bf2a638f4bc35f