change getBounds to return 0000 iff there are zero points

change getBounds to return 0000 iff there are zero points

This is a contract change for SkPath::getBounds(), which formally was defined to return 0,0,0,0 for a 1-point path, regardless of the coordinates of that point. This seems wacky/inconsistent, and was causing other bugs (incorrect bounds) when this was unioned with other rects.

Does anyone remember why we defined it this way?

BUG=513799
Committed: https://skia.googlesource.com/skia/+/91f283bb4e6ea71bbd4e6efc27befc29118ee543

[reed1, 2015-07-27 21:06:25]

reed@google.com changed reviewers:
+ caryclark@google.com, fmalita@chromium.org, junov@google.com,
schenney@google.com

[reed1, 2015-07-27 21:07:50]

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

[commit-bot: I haz the power, 2015-07-27 21:07:56]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1261773002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1261773002/1

[commit-bot: I haz the power, 2015-07-27 21:15:40]

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

[commit-bot: I haz the power, 2015-07-27 21:15:40]

Dry run: This issue passed the CQ dry run.

[reed1, 2015-07-27 21:52:08]

ptal

[f(malita), 2015-07-27 21:59:14]

On 2015/07/27 21:52:08, reed1 wrote:
> ptal

Not familiar with the original motivation, but this LGTM.

[caryclark, 2015-07-28 12:11:18]

lgtm

[reed1, 2015-07-28 12:59:52]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2015-07-28 13:00:05]

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

[commit-bot: I haz the power, 2015-07-28 13:00:52]

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