It is dangerous to ignore SkRect::intersect's return value

It is dangerous to ignore SkRect::intersect's return value

Committed: https://skia.googlesource.com/skia/+/152f524fd325b7776b01f84afbfe2fa071648a05

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

[robertphillips, 2015-01-02 15:47:07]

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

[robertphillips, 2015-01-02 15:47:11]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2015-01-02 15:48:11]

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

[commit-bot: I haz the power, 2015-01-02 15:48:12]

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-01-02 21:48 UTC

[commit-bot: I haz the power, 2015-01-02 15:48:16]

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

[commit-bot: I haz the power, 2015-01-02 15:48:16]

Presubmit check for 833943002-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Since the CL is editing public API, you must have an LGTM from one of:
('reed@chromium.org', 'reed@google.com', 'bsalomon@chromium.org',
'bsalomon@google.com', 'djsollen@chromium.org', 'djsollen@google.com')

[robertphillips, 2015-01-02 18:07:39]

Hmmm - not really in love with this CL. Thoughts?

[reed1, 2015-01-05 14:16:33]

yummy
lgtm

[robertphillips, 2015-01-05 16:22:01]

This is paused until https://codereview.chromium.org/814123003/ (Add handling
for Skia's SkPath::intersect's return value) can land in Chromium.

[robertphillips, 2015-01-07 16:51:08]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2015-01-07 16:51:23]

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

[commit-bot: I haz the power, 2015-01-07 17:01:16]

Committed patchset #6 (id:100001) as
https://skia.googlesource.com/skia/+/152f524fd325b7776b01f84afbfe2fa071648a05

[robertphillips, 2015-01-07 17:11:49]

A revert of this CL (patchset #6 id:100001) has been created in
https://codereview.chromium.org/825983005/ by robertphillips@google.com.

The reason for reverting is: Still more Chromium clean up to do.

[robertphillips, 2015-01-07 20:15:27]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2015-01-07 20:16:04]

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

[commit-bot: I haz the power, 2015-01-07 20:16:14]

Committed patchset #6 (id:100001) as
https://skia.googlesource.com/skia/+/f4e5995ac70d4614e0a05b92a8a03e2b7d76bd9c