Bug 43086
| Summary: | fast/canvas/canvas-ImageData-behaviour.html fails on chromium after r64156 | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Victor Wang <victorw> |
| Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | jamesr, levin, senorblanco, victorw |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | PC | ||
| OS: | All | ||
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
...
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
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.