Add GM to test non-AA pixel snapping for points, lines, rects.

Add GM to test non-AA pixel snapping for points, lines, rects.

Committed: https://skia.googlesource.com/skia/+/87cbcf3f86a58cacab2e576ebc0e29399a1d48b3

[bsalomon, 2015-04-22 14:52:14]

bsalomon@google.com changed reviewers:
+ robertphillips@google.com

[bsalomon, 2015-04-22 14:52:36]

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

[commit-bot: I haz the power, 2015-04-22 14:52:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1070213003/40001

[commit-bot: I haz the power, 2015-04-22 15:00:57]

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

[commit-bot: I haz the power, 2015-04-22 15:00:58]

Dry run: Try jobs failed on following builders:
  Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Debug-Trybot on client.skia (JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)
  Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-Shared-Trybot on client.skia
(JOB_FAILED,
http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2...)

[bsalomon, 2015-04-22 15:23:27]

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

[commit-bot: I haz the power, 2015-04-22 15:23:36]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1070213003/60001

[robertphillips, 2015-04-22 15:28:12]

lgtm + nits

https://codereview.chromium.org/1070213003/diff/40001/gm/pixelsnap.cpp
File gm/pixelsnap.cpp (right):

https://codereview.chromium.org/1070213003/diff/40001/gm/pixelsnap.cpp#newcode33
gm/pixelsnap.cpp:33: int yoffset = SkScalarCeilToInt(labelPaint.getTextSize());
"+( " -> "+ (" ?

https://codereview.chromium.org/1070213003/diff/40001/gm/pixelsnap.cpp#newcod...
gm/pixelsnap.cpp:113: private:
skiagm::GM ?

https://codereview.chromium.org/1070213003/diff/40001/gm/pixelsnap.cpp#newcod...
gm/pixelsnap.cpp:121: 
private:
    PixelSnapGM ?

[commit-bot: I haz the power, 2015-04-22 15:28:49]

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

[commit-bot: I haz the power, 2015-04-22 15:28:50]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2015-04-22 15:45:56]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2015-04-22 15:45:56]

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

[bsalomon, 2015-04-22 15:46:02]

https://codereview.chromium.org/1070213003/diff/40001/gm/pixelsnap.cpp
File gm/pixelsnap.cpp (right):

https://codereview.chromium.org/1070213003/diff/40001/gm/pixelsnap.cpp#newcode33
gm/pixelsnap.cpp:33: int yoffset = SkScalarCeilToInt(labelPaint.getTextSize());
On 2015/04/22 15:28:11, robertphillips wrote:
> "+( " -> "+ (" ?

Done.

https://codereview.chromium.org/1070213003/diff/40001/gm/pixelsnap.cpp#newcod...
gm/pixelsnap.cpp:113: private:
On 2015/04/22 15:28:11, robertphillips wrote:
> skiagm::GM ?

Done.

https://codereview.chromium.org/1070213003/diff/40001/gm/pixelsnap.cpp#newcod...
gm/pixelsnap.cpp:121: 
On 2015/04/22 15:28:11, robertphillips wrote:
> private:
>     PixelSnapGM ?

Done.

[commit-bot: I haz the power, 2015-04-22 15:46:08]

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

[commit-bot: I haz the power, 2015-04-22 15:51:44]

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