add gm for path-arcs, and catch degenerate arc in conic-case

add gm for path-arcs, and catch degenerate arc in conic-case

BUG=skia:3428
Committed: https://skia.googlesource.com/skia/+/9e779d495130009926fc5394a8971eec20494e5f

[reed2, 2015-02-17 13:27:33]

reed@chromium.org changed reviewers:
+ fmalita@chromium.org, robertphillips@google.com

[reed2, 2015-02-17 13:27:34]

should fix layouttests

[reed2, 2015-02-17 13:27:38]

The CQ bit was checked by reed@chromium.org

[commit-bot: I haz the power, 2015-02-17 13:27:48]

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

[commit-bot: I haz the power, 2015-02-17 13:27:49]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2015-02-17 19:27 UTC

[commit-bot: I haz the power, 2015-02-17 13:33:37]

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

[commit-bot: I haz the power, 2015-02-17 13:33:38]

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

[f(malita), 2015-02-17 13:45:26]

lgtm % win build fix

Let's also add a bug number to link this to
https://code.google.com/p/skia/issues/detail?id=3428

[robertphillips, 2015-02-17 17:33:45]

lgtm + nits

https://codereview.chromium.org/931183002/diff/1/gm/addarc.cpp
File gm/addarc.cpp (right):

https://codereview.chromium.org/931183002/diff/1/gm/addarc.cpp#newcode174
gm/addarc.cpp:174: 
Which one?

https://codereview.chromium.org/931183002/diff/1/gm/addarc.cpp#newcode217
gm/addarc.cpp:217: path.moveTo(0, 2);
overlength ?

[reed2, 2015-02-17 18:31:21]

On 2015/02/17 13:45:26, f(malita) wrote:
> lgtm % win build fix
> 
> Let's also add a bug number to link this to
> https://code.google.com/p/skia/issues/detail?id=3428

Done

[reed2, 2015-02-17 18:35:36]

https://codereview.chromium.org/931183002/diff/1/gm/addarc.cpp
File gm/addarc.cpp (right):

https://codereview.chromium.org/931183002/diff/1/gm/addarc.cpp#newcode174
gm/addarc.cpp:174: 
On 2015/02/17 17:33:45, robertphillips wrote:
> Which one?

Done.

https://codereview.chromium.org/931183002/diff/1/gm/addarc.cpp#newcode217
gm/addarc.cpp:217: path.moveTo(0, 2);
On 2015/02/17 17:33:45, robertphillips wrote:
> overlength ?

Not sure what you mean.

[reed2, 2015-02-17 18:36:01]

New patchsets have been uploaded after l-g-t-m from
fmalita@chromium.org,robertphillips@google.com

[reed2, 2015-02-17 18:55:20]

https://codereview.chromium.org/931183002/diff/1/gm/addarc.cpp
File gm/addarc.cpp (right):

https://codereview.chromium.org/931183002/diff/1/gm/addarc.cpp#newcode217
gm/addarc.cpp:217: path.moveTo(0, 2);
On 2015/02/17 17:33:45, robertphillips wrote:
> overlength ?

Done.

[reed2, 2015-02-17 18:55:25]

The CQ bit was checked by reed@chromium.org

[commit-bot: I haz the power, 2015-02-17 18:56:28]

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

[commit-bot: I haz the power, 2015-02-17 19:03:28]

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

[commit-bot: I haz the power, 2015-02-17 19:03:29]

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

[reed2, 2015-02-17 19:08:25]

The CQ bit was checked by reed@chromium.org

[commit-bot: I haz the power, 2015-02-17 19:08:29]

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

[commit-bot: I haz the power, 2015-02-17 19:10:10]

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

[commit-bot: I haz the power, 2015-02-17 19:10:11]

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

[reed2, 2015-02-17 19:15:51]

The CQ bit was checked by reed@chromium.org

[commit-bot: I haz the power, 2015-02-17 19:16:39]

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

[commit-bot: I haz the power, 2015-02-17 19:43:18]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/9e779d495130009926fc5394a8971eec20494e5f