cull edges that are to the right of the clip

cull edges that are to the right of the clip

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/31223e0cb74f47f63b094520a9830c525b72fe87

[reed1, 2015-02-09 16:00:03]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2015-02-09 16:00:45]

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

[commit-bot: I haz the power, 2015-02-09 16:00:46]

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-09 22:00 UTC

[reed1, 2015-02-09 16:01:57]

The CQ bit was unchecked by reed@google.com

[reed1, 2015-02-09 16:03:06]

reed@google.com changed reviewers:
+ caryclark@google.com

[reed1, 2015-02-09 16:03:06]

The interesting change is in SkScan_Path.cpp

[caryclark, 2015-02-09 16:06:30]

lgtm

https://codereview.chromium.org/913503002/diff/1/src/core/SkScan_Path.cpp
File src/core/SkScan_Path.cpp (right):

https://codereview.chromium.org/913503002/diff/1/src/core/SkScan_Path.cpp#new...
src/core/SkScan_Path.cpp:443: if (0 == count) {
I don't mind this pattern -- but is it preferable to 

       if (!count) {

?

[reed1, 2015-02-09 16:08:44]

On 2015/02/09 16:06:30, caryclark wrote:
> lgtm
> 
> https://codereview.chromium.org/913503002/diff/1/src/core/SkScan_Path.cpp
> File src/core/SkScan_Path.cpp (right):
> 
>
https://codereview.chromium.org/913503002/diff/1/src/core/SkScan_Path.cpp#new...
> src/core/SkScan_Path.cpp:443: if (0 == count) {
> I don't mind this pattern -- but is it preferable to 
> 
>        if (!count) {
> 
> ?

We seem to have gobs of both. I happen to find this form a tiny bit more clear,
as I find ! to be very narrow and easier to miss next to the (, but I'm not
suggesting we don't use it :)

[reed1, 2015-02-09 16:31:59]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2015-02-09 16:32:57]

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

[commit-bot: I haz the power, 2015-02-09 16:33:10]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/31223e0cb74f47f63b094520a9830c525b72fe87