Bug 58309 – [GTK] fast/canvas/DrawImageSinglePixelStretch.html fails

• Status: RESOLVED
• Resolution: FIXED
• Component: Canvas

[Philippe Normand]

The color inside the rectangle is not uniform, it's more like a gradient.

Will skip for now.

[Matthew Delaney]

Yea, that's the incorrect behavior you're seeing then. It does show a 'FAILED', correct?

I've interpreted this portion of the spec as the inspiration for this test:
"If the original image data is a bitmap image, the value painted at a point in the destination rectangle is computed by filtering the original image data. The user agent may use any filtering algorithm (for example bilinear interpolation or nearest-neighbor). When the filtering algorithm requires a pixel value from outside the original image data, it must instead use the value from the nearest edge pixel. (That is, the filter uses 'clamp-to-edge' behavior.)"

It also came to my attention that many rendering platforms don't usually do this behavior of clamp to edge because it's not desired in cases where tiling of an image happens and such. Though, the pixels can be first copied into a buffer image and then drawn to effectively create the clamp to edge behavior.

[Nishant Bansal]

The same issue is also present in EFL port.

[Dirk Schulze]

Can you upload and expected and actual image for comparison please?

[Martin Robinson]

Created attachment 189642 [details]
Patch

[Martin Robinson]

(In reply to comment #3)
> Can you upload and expected and actual image for comparison please?

The test is a text dump, but with a bit of tinkering I was able to see the canvas. It's clear that the issue is that we are sampling outside the source rectangle. I've uploaded a patch fixing the issue.

[Nishant Bansal]

I tried this patch. The issue is still present in EFL port.

[Martin Robinson]

(In reply to comment #6)
> I tried this patch. The issue is still present in EFL port.

What version of Cairo? Also, are you using a backend other than image?

[Nishant Bansal]

(In reply to comment #7)
> (In reply to comment #6)
> > I tried this patch. The issue is still present in EFL port.
> 
> What version of Cairo? Also, are you using a backend other than image?

cairo version is 1.12.4.
I didn't understand your second query.

[Nishant Bansal]

There was some build problem or something. I tried this patch on the latest webkit and it worked fine.

[Martin Robinson]

(In reply to comment #9)
> There was some build problem or something. I tried this patch on the latest webkit and it worked fine.

Thanks for the information. Dirk, maybe you would be willing to review this patch?

[Alejandro G. Castro]

Comment on attachment 189642 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=189642&action=review

LGTM!

> Source/WebCore/platform/graphics/cairo/PlatformContextCairo.cpp:196
> +    // The pattern transformation properly scales the pattern for when the source rectangle is a

Nit, "for when" -> "when"

[Martin Robinson]

Comment on attachment 189642 [details]
Patch

Clearing flags on attachment: 189642

Committed r148681: <http://trac.webkit.org/changeset/148681>

[Martin Robinson]

All reviewed patches have been landed.  Closing bug.