interpret conic tolerance correctly -- big rebaseline

interpret conic tolerance correctly -- big rebaseline

need this to land in chrome
https://codereview.chromium.org/918233002/

NOTREECHECKS=True

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

[reed1, 2015-02-11 18:54:49]

reed@google.com changed reviewers:
+ caryclark@google.com, robertphillips@google.com, stephana@google.com

[reed1, 2015-02-11 18:54:50]

This is a giant rebaseliner -- ~75 different GMs are affected.

A few places improved, nothing (to my eye) got worse).

Real motivation (besides the old logic was just stupid) is to make tolerance
real so we can tweak it when stroking tiny things that have a known CTM...
(future CL)

[_cary, 2015-02-11 19:12:26]

caryclark@chromium.org changed reviewers:
+ caryclark@chromium.org

[_cary, 2015-02-11 19:12:26]

lgtm

[reed1, 2015-02-11 19:15:46]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2015-02-11 19:16:36]

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

[commit-bot: I haz the power, 2015-02-11 19:16:37]

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-12 01:16 UTC

[reed1, 2015-02-11 19:17:00]

The CQ bit was unchecked by reed@google.com

[robertphillips, 2015-02-11 20:01:09]

lgtm

https://codereview.chromium.org/915103002/diff/20001/src/core/SkGeometry.cpp
File src/core/SkGeometry.cpp (right):

https://codereview.chromium.org/915103002/diff/20001/src/core/SkGeometry.cpp#...
src/core/SkGeometry.cpp:1441: 
suggested?

[robertphillips, 2015-02-11 20:01:54]

https://codereview.chromium.org/915103002/diff/20001/src/core/SkGeometry.cpp
File src/core/SkGeometry.cpp (right):

https://codereview.chromium.org/915103002/diff/20001/src/core/SkGeometry.cpp#...
src/core/SkGeometry.cpp:1451: 
Don't you want to put this in a gypi somewhere?

[reed1, 2015-02-11 21:59:51]

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

[reed1, 2015-02-12 15:59:15]

On 2015/02/11 20:01:54, robertphillips wrote:
> https://codereview.chromium.org/915103002/diff/20001/src/core/SkGeometry.cpp
> File src/core/SkGeometry.cpp (right):
> 
>
https://codereview.chromium.org/915103002/diff/20001/src/core/SkGeometry.cpp#...
> src/core/SkGeometry.cpp:1451: 
> Don't you want to put this in a gypi somewhere?

Will land it in chrome today

[reed2, 2015-02-13 01:47:19]

The CQ bit was checked by reed@chromium.org

[commit-bot: I haz the power, 2015-02-13 01:47:25]

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

[reed2, 2015-02-13 01:56:02]

The CQ bit was unchecked by reed@chromium.org

[reed2, 2015-02-13 01:56:08]

The CQ bit was checked by reed@chromium.org

[commit-bot: I haz the power, 2015-02-13 01:56:16]

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

[commit-bot: I haz the power, 2015-02-13 01:56:24]

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