Move GrAtlasTextBatch blob regeneration to template

Move GrAtlasTextBatch blob regeneration to template

This will remove the runtime branches.  I don't think it makes the
code any harder to read either

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/60ce86d4718dab83f33488ec41710ad6763fc7f8

[joshualitt, 2015-11-23 17:43:34]

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

[joshualitt, 2015-11-23 17:43:34]

I think, on the whole, this is easier to read, and it should definitely be more
efficient.  I am open to other ways to do this though.

Brian mostly added just FYI

[robertphillips, 2015-11-23 18:42:52]

https://codereview.chromium.org/1466333003/diff/60001/src/gpu/batches/GrAtlas...
File src/gpu/batches/GrAtlasTextBatch.cpp (right):

https://codereview.chromium.org/1466333003/diff/60001/src/gpu/batches/GrAtlas...
src/gpu/batches/GrAtlasTextBatch.cpp:29: if (regenTexCoords) {
mv width & height in here ?

https://codereview.chromium.org/1466333003/diff/60001/src/gpu/batches/GrAtlas...
src/gpu/batches/GrAtlasTextBatch.cpp:46: 
won'd -> won't ?

https://codereview.chromium.org/1466333003/diff/60001/src/gpu/batches/GrAtlas...
src/gpu/batches/GrAtlasTextBatch.cpp:128: template <bool regenPos, bool
regenCol, bool regenTexCoords, bool regenGlyphs>
That's a lot of parameters ...

https://codereview.chromium.org/1466333003/diff/60001/src/gpu/batches/GrAtlas...
src/gpu/batches/GrAtlasTextBatch.cpp:406: 
So we go from Booleans to a mask so we can go through a switch statement to
expand back out to Booleans ?

[robertphillips, 2015-11-23 20:10:34]

lgtm

[bsalomon, 2015-11-23 20:11:38]

lgtm

[jvanverth1, 2015-11-23 20:11:59]

On 2015/11/23 20:11:38, bsalomon wrote:
> lgtm

Lgtm too

[joshualitt, 2015-11-23 20:35:30]

The CQ bit was checked by joshualitt@google.com

[commit-bot: I haz the power, 2015-11-23 20:35:52]

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

[joshualitt, 2015-11-23 20:44:55]

The CQ bit was unchecked by joshualitt@google.com

[joshualitt, 2015-11-23 20:53:19]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-11-23 20:53:19]

The patchset sent to the CQ was uploaded after l-g-t-m from
jvanverth@google.com, bsalomon@google.com, robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1466333003/#ps140001
(title: "add inline qualifier")

[commit-bot: I haz the power, 2015-11-23 20:53:28]

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

[commit-bot: I haz the power, 2015-11-23 21:08:25]

Committed patchset #8 (id:140001) as
https://skia.googlesource.com/skia/+/60ce86d4718dab83f33488ec41710ad6763fc7f8