Fix SkRRect isEmpty assert

Fix SkRRect isEmpty assert

The rectangle could be empty but not all zeroes.

BUG=skia:2379
Signed-off-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com>;

Committed: http://code.google.com/p/skia/source/detail?r=14041

[robertphillips, 2014-04-03 12:35:55]

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

https://codereview.chromium.org/223213005/diff/1/src/core/SkRRect.cpp#newcode466
src/core/SkRRect.cpp:466: 
I don't think this is required at all. The preceding fRect.isEmpty is already
checking this.

[siglesias, 2014-04-03 12:44:11]

On 2014/04/03 12:35:55, robertphillips wrote:
> https://codereview.chromium.org/223213005/diff/1/src/core/SkRRect.cpp
> File src/core/SkRRect.cpp (right):
> 
>
https://codereview.chromium.org/223213005/diff/1/src/core/SkRRect.cpp#newcode466
> src/core/SkRRect.cpp:466: 
> I don't think this is required at all. The preceding fRect.isEmpty is already
> checking this.

You are right. So removing the ASSERT would make the code still working as
expected and fix this issue.

I'm going to upload the patch accordingly.

[siglesias, 2014-04-03 12:52:56]

Remove the ASSERT

[robertphillips, 2014-04-03 13:05:22]

lgtm

[robertphillips, 2014-04-03 13:05:37]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2014-04-03 13:05:41]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/siglesias@igalia.com/223213005/20001

[commit-bot: I haz the power, 2014-04-03 14:35:49]

Change committed as 14041