Add device space skshader GM to test kDevice_GrCoordSet

Add device space skshader GM to test kDevice_GrCoordSet

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

[bsalomon, 2014-12-22 22:24:33]

bsalomon@google.com changed reviewers:
+ joshualitt@google.com, reed@google.com

[bsalomon, 2014-12-22 22:24:34]

reed@ for the blitter changes.

[joshualitt, 2014-12-23 02:27:59]

On 2014/12/22 22:24:34, bsalomon wrote:
> reed@ for the blitter changes.

lgtm

[joshualitt, 2014-12-23 02:31:51]

actually, one nit and one comment

https://codereview.chromium.org/816003002/diff/40001/gm/dcshader.cpp
File gm/dcshader.cpp (right):

https://codereview.chromium.org/816003002/diff/40001/gm/dcshader.cpp#newcode206
gm/dcshader.cpp:206: SkDebugf("Could not find Funkster.ttf, please set
--resourcePath correctly.\n");
nit, but you could line wrap

https://codereview.chromium.org/816003002/diff/40001/gm/dcshader.cpp#newcode239
gm/dcshader.cpp:239: viewMats.back().postScale(0.5f, 1.1f);
I know the shader ignores perspective, but it might be worth adding a
perspective matrix anyways because it will exercise a bit more of the backend.

[bsalomon, 2014-12-23 20:34:23]

I added a perspective device coord matrix and updated the shader to use
ensureFSCoords2D() to draw correctly. Is that what you meant?

[joshualitt, 2014-12-23 20:37:37]

On 2014/12/23 20:34:23, bsalomon wrote:
> I added a perspective device coord matrix and updated the shader to use
> ensureFSCoords2D() to draw correctly. Is that what you meant?

perfect, yes. Thanks!

[bsalomon, 2014-12-29 14:12:50]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2014-12-29 14:13:54]

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

[commit-bot: I haz the power, 2014-12-29 14:15:34]

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

[commit-bot: I haz the power, 2014-12-29 14:15:35]

Try jobs failed on following builders:
  Test-Ubuntu13.10-GCE-NoGPU-x86_64-Debug-Trybot on client.skia
(http://build.chromium.org/p/client.skia/builders/Test-Ubuntu13.10-GCE-NoGPU-x...)

[bsalomon, 2014-12-29 14:45:36]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2014-12-29 14:46:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/816003002/100001

[commit-bot: I haz the power, 2014-12-29 14:47:44]

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

[commit-bot: I haz the power, 2014-12-29 14:47:45]

Try jobs failed on following builders:
  Test-Ubuntu13.10-GCE-NoGPU-x86_64-Debug-Trybot on client.skia
(http://build.chromium.org/p/client.skia/builders/Test-Ubuntu13.10-GCE-NoGPU-x...)

[bsalomon, 2014-12-29 14:55:56]

The CQ bit was checked by bsalomon@google.com

[commit-bot: I haz the power, 2014-12-29 14:56:04]

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

[commit-bot: I haz the power, 2014-12-29 15:05:31]

Committed patchset #7 (id:120001) as
https://skia.googlesource.com/skia/+/b0ae649b7ebda86ef53bf913798b809d500973ed