Bug 43086 – fast/canvas/canvas-ImageData-behaviour.html fails on chromium after r64156

• Status: RESOLVED
• Resolution: WONTFIX
• Component: WebKit Misc.

[Victor Wang]

fast/canvas/canvas-ImageData-behaviour.html fails on chromium after r64156. Looks like data type and rounding issue.

Here are the example of actual output on chromium win:

PASS imageData.width is 2
...
FAIL imageData.data[0] = -1, imageData.data[0] should be 255. Was 0.
PASS imageData.data[0] = "0", imageData.data[0] is 0
...
FAIL imageData.data[0] = Infinity, imageData.data[0] should be 0. Was 255.
PASS imageData.data[0] = -Infinity, imageData.data[0] is 0
FAIL imageData.data[0] = -5, imageData.data[0] should be 251. Was 0.
PASS imageData.data[0] = -0.5, imageData.data[0] is 0
PASS imageData.data[0] = 0, imageData.data[0] is 0
FAIL imageData.data[0] = 0.5, imageData.data[0] should be 0. Was 1.
PASS imageData.data[0] = 5, imageData.data[0] is 5
PASS imageData.data[0] = 2147483647, imageData.data[0] is 255
FAIL imageData.data[0] = 2147483648, imageData.data[0] should be 0. Was 255.
FAIL imageData.data[0] = 2147483649, imageData.data[0] should be 1. Was 255.
FAIL imageData.data[0] = -1099511627773.5, imageData.data[0] should be 3. Was 0.
FAIL imageData.data[0] = 9007199254740990, imageData.data[0] should be 254. Was 255.
PASS imageData.data[0] = 5.4, imageData.data[0] is 5
PASS imageData.data[0] = 255, imageData.data[0] is 255
FAIL imageData.data[0] = 256, imageData.data[0] should be 0. Was 255.
PASS imageData.data[0] = null, imageData.data[0] is 0
...

[Victor Wang]

This is because the patch http://trac.webkit.org/changeset/64156/trunk trunks and wraps the value between 0-255 in JavaScriptCore.

[Victor Wang]

(In reply to comment #1)
> This is because the patch http://trac.webkit.org/changeset/64156/trunk trunks and wraps the value between 0-255 in JavaScriptCore.

Should we do the same thing (truncate and wrap value to 0-255) like this? cced more people...

[Stephen Chenney]

LayoutTest failures for Chromium are being marked WontFix. The Bug is still accessible and referenced from TestExpectations.