Should not use clipOutRoundedRect when a given rounded rect is empty.

Should not use clipOutRoundedRect when a given rounded rect is empty.

BUG=303922, 303383
TEST=fast/borders/border-radius-with-empty-inner-border.html,fast/borders/empty-border-radius.html

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=159641

[Stephen Chennney, 2013-10-04 17:47:12]

The change seems fine, but why not produce correct references for the new tests?
Are you planning to change them to pixel tests? If so, I don't think the
rebaseline bot will help you unless you go through the TestExpectations route.

[tasak, 2013-10-07 06:42:24]

Thank you for reviewing.

On 2013/10/04 17:47:12, Stephen Chenney wrote:
> The change seems fine, but why not produce correct references for the new
tests?
> Are you planning to change them to pixel tests? If so, I don't think the
> rebaseline bot will help you unless you go through the TestExpectations route.

I see.
I changed the layout tests to use pixel tests.

[tasak, 2013-10-11 04:19:59]

Would you review this CL?

[eseidel, 2013-10-11 04:51:45]

lgtm

It seems the clip-out functiopn could also just ignore the empty rect instead of
the caller needing to.

[tasak, 2013-10-15 02:27:59]

Thank you for reviewing.

On 2013/10/11 04:51:45, eseidel wrote:
> lgtm
> 
> It seems the clip-out functiopn could also just ignore the empty rect instead
of
> the caller needing to.

Yes. However, the clip-out function just calls skia's API. I guess, this issue
is caused by skia update.

[commit-bot: I haz the power, 2013-10-15 02:28:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tasak@google.com/25977004/25001

[commit-bot: I haz the power, 2013-10-15 05:01:02]

Change committed as 159641