path effect on a very thin line should not be discarded

path effect on a very thin line should not be discarded

This patch fixes the second line rendering in SampleApp:Slides.

BUG=
TEST=SampleApp Slides

[guanqun, 2012-02-12 10:57:59]

1. I've checked the gm output before and after this CL, there's no diff. So I
think it's OK, no regression for gm found.

2. Should I port "Slides" in SampleApp to gm?

Please review. Thanks.

[bsalomon, 2012-02-13 14:28:47]

On 2012/02/12 10:57:59, guanqun wrote:
> 1. I've checked the gm output before and after this CL, there's no diff. So I
> think it's OK, no regression for gm found.
> 
> 2. Should I port "Slides" in SampleApp to gm?
> 
> Please review. Thanks.

It seems right to me. Mike?

[reed1, 2012-02-13 14:37:57]

lgtm

good catch. Does this work fine in the raster case, and was only a problem for
gpu?

[bsalomon, 2012-02-13 14:43:22]

On 2012/02/13 14:28:47, bsalomon wrote:
> On 2012/02/12 10:57:59, guanqun wrote:
> > 1. I've checked the gm output before and after this CL, there's no diff. So
I
> > think it's OK, no regression for gm found.
> > 
> > 2. Should I port "Slides" in SampleApp to gm?

It'd be great to catch the use case in a GM somehow. IIRC the slides sample
actually cycles through different screens each time it is clicked. So it may be
easier to create a new GM with different path effects.

[guanqun, 2012-02-13 14:44:21]

On 2012/02/13 14:37:57, reed1 wrote:
> good catch. Does this work fine in the raster case, and was only a problem for
> gpu?

Raster case works fine, it was a problem for gpu.

[bsalomon, 2012-02-13 14:49:31]

Landed the change at r3171

[guanqun, 2012-02-13 15:12:48]

On 2012/02/13 14:49:31, bsalomon wrote:
> Landed the change at r3171

Thanks for landing this CL. But I've just uploaded a second pathset, adding this
gm test, it's ported from SampleApp:Slides, should I upload another request?

[bsalomon, 2012-02-13 15:18:57]

On 2012/02/13 15:12:48, guanqun wrote:
> On 2012/02/13 14:49:31, bsalomon wrote:
> > Landed the change at r3171
> 
> Thanks for landing this CL. But I've just uploaded a second pathset, adding
this
> gm test, it's ported from SampleApp:Slides, should I upload another request?

Thanks for the GM! I just landed it at r3172.