Fix bound returned by SkPath::isRect when the path contains a trailing moveTo
authorrobertphillips <robertphillips@google.com>
Mon, 29 Dec 2014 19:36:39 +0000 (11:36 -0800)
committerCommit bot <commit-bot@chromium.org>
Mon, 29 Dec 2014 19:36:39 +0000 (11:36 -0800)
commitfe7c427e3d1c2c98bce7a3fa0ae6b5864527f169
tree3bb31eeeb6bd859e3a6a23c6c57c879d4ad65391
parent40d4bd8dafe9ed844f29a3de3d8965672248ee73
Fix bound returned by SkPath::isRect when the path contains a trailing moveTo

Oddly enough this was fixed in:

https://codereview.chromium.org/16950021/ (add rect-output parameter to isRect, allowing us to return the correct bounds even if a rectagular path has a trailing moveTo)

but was reverted here:

https://skia.googlesource.com/skia/+/8fd160350ca5f57fbb1b2e03383c5778414a9b48

since it appeared to be crashing Chrome's trybots. I think it just fell through the cracks after that.

If this sticks I will land a follow on patch for the stroke issue reported in the original bug (crbug.com/247770).

BUG=247770,445368

Review URL: https://codereview.chromium.org/834483002
src/core/SkPath.cpp
tests/PathTest.cpp