Simplify nvpr text

Simplify nvpr text

- Drops device-space glyphs in favor of perf/simplicity.

- Removes residual complexities that would flip glyphs vertically for
  compatibility with nvpr glyph loading, which is no longer used.

- Drops hairline support since they required new paths for every draw
  matrix, and could only be supported under certain circumstances.

- Quits checking for color bitmap fonts in canDrawText since the
  normal color emoji fallback will handle them anyway.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/7d5c950e85b4766a05ed834c594033c9dd6913cb

[Chris Dalton, 2015-10-01 04:46:42]

cdalton@nvidia.com changed reviewers:
+ bsalomon@google.com, kkinnunen@nvidia.com

[Chris Dalton, 2015-10-01 04:46:43]

I'm curious how you guys feel about this. I like getting drawText and
drawPosText to produce similar results, but the biggest problem with this
approach is that drawText can now get drift compared to how another text context
might decide to position the glyphs. Is this a problem?

[Mark Kilgard, 2015-10-01 17:33:24]

markkilgard@gmail.com changed reviewers:
+ markkilgard@gmail.com

[Mark Kilgard, 2015-10-01 17:33:25]

lgtm

mostly clean up, eliminating code

https://chromiumcodereview.appspot.com/1380973002/diff/1/samplecode/SampleApp...
File samplecode/SampleApp.cpp (right):

https://chromiumcodereview.appspot.com/1380973002/diff/1/samplecode/SampleApp...
samplecode/SampleApp.cpp:246: fCurIntf =
SkRef/*GrGLInterfaceRemoveNVPR*/(glInterface.get());
would it be good to have a command line to SampleApp that decided this?

could there even be a keyboard key to toggle this?

I think yes

[bsalomon, 2015-10-02 13:22:07]

lgtm

[Chris Dalton, 2015-10-03 16:39:21]

https://codereview.chromium.org/1380973002/diff/1/samplecode/SampleApp.cpp
File samplecode/SampleApp.cpp (right):

https://codereview.chromium.org/1380973002/diff/1/samplecode/SampleApp.cpp#ne...
samplecode/SampleApp.cpp:246: fCurIntf =
SkRef/*GrGLInterfaceRemoveNVPR*/(glInterface.get());
On 2015/10/01 17:33:24, Mark Kilgard wrote:
> would it be good to have a command line to SampleApp that decided this?
> 
> could there even be a keyboard key to toggle this?
> 
> I think yes

Oops, that was an accidental hack that found its way into this patch. Updating
SampleApp to support nvpr would be nice, but probably something for a different
CL.

[Chris Dalton, 2015-10-03 16:57:04]

The CQ bit was checked by cdalton@nvidia.com

[Chris Dalton, 2015-10-03 16:57:04]

The patchset sent to the CQ was uploaded after l-g-t-m from bsalomon@google.com,
markkilgard@gmail.com
Link to the patchset: https://codereview.chromium.org/1380973002/#ps40001
(title: "Remove accidental SampleApp hack")

[commit-bot: I haz the power, 2015-10-03 16:57:10]

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

[commit-bot: I haz the power, 2015-10-03 16:57:11]

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

[commit-bot: I haz the power, 2015-10-03 16:57:12]

The author cdalton@nvidia.com has not signed Google Contributor License
Agreement. Please visit https://cla.developers.google.com to sign and manage
CLA.

[Chris Dalton, 2015-10-03 20:21:44]

The CQ bit was checked by cdalton@nvidia.com

[commit-bot: I haz the power, 2015-10-03 20:21:51]

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

[commit-bot: I haz the power, 2015-10-03 20:28:42]

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