remove contains(x,y) for rects and rrects ... not well defined, and unused

remove contains(x,y) for rects and rrects ... not well defined, and unused

BUG=
R=robertphillips@google.com

Committed: https://code.google.com/p/skia/source/detail?r=12022

[robertphillips, 2013-10-30 17:18:48]

lgtm + somewhat important question

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

https://codereview.chromium.org/51953003/diff/1/src/core/SkPath.cpp#newcode2964
src/core/SkPath.cpp:2964: static bool contains_inclusive(const SkRect& r,
SkScalar x, SkScalar y) {
shouldn't that be "<= r.fBottom"?

[reed1, 2013-10-30 17:38:53]

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

https://codereview.chromium.org/51953003/diff/1/src/core/SkPath.cpp#newcode2964
src/core/SkPath.cpp:2964: static bool contains_inclusive(const SkRect& r,
SkScalar x, SkScalar y) {
On 2013/10/30 17:18:48, robertphillips wrote:
> shouldn't that be "<= r.fBottom"?

doh! good catch. done.

[reed1, 2013-10-30 17:39:49]

Committed patchset #1 manually as r12022 (presubmit successful).

[zac, 2014-02-06 23:06:29]

On 2013/10/30 17:39:49, reed1 wrote:
> Committed patchset #1 manually as r12022 (presubmit successful).

We were. :-( update skia in our platform and it went missing. oh well. easy to
replace.