RESOLVED FIXED 46557
[Chromium] animations/opacity-transform-animation.html fails on Chromium 
https://bugs.webkit.org/show_bug.cgi?id=46557
Summary [Chromium] animations/opacity-transform-animation.html fails on Chromium
Mihai Parparita
Reported 2010-09-24 23:51:05 PDT
The test was added in http://trac.webkit.org/changeset/68323, and it's been getting image diffs since then. The animation is subtle enough (going from opacity .8 to .9) that I'm not sure if it's just a slight color difference in Chromium or an actual behavior difference.
Attachments
Patch (6.28 KB, patch)
2010-09-27 23:28 PDT, Mihai Parparita 		no flags
DetailsFormatted DiffDiff
Patch (7.60 KB, patch)
2010-09-28 00:10 PDT, Mihai Parparita 		no flags
DetailsFormatted DiffDiff
Patch (3.82 KB, patch)
2010-09-28 18:11 PDT, Mihai Parparita 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Mihai Parparita
Comment 1 2010-09-27 23:28:31 PDT
Created attachment 69024 [details] Patch
Mihai Parparita
Comment 2 2010-09-27 23:30:36 PDT
Not 100% sure why the more subtle opacity change caused image diffs with the Chromium DRT (precision issues?), but since the intent of the test is to test two properties changing (not image fidelity), it still seems valid with this change.
Mihai Parparita
Comment 3 2010-09-27 23:41:42 PDT
Comment on attachment 69024 [details] Patch Nevermind about this patch, I wasn't actually passing --chromium to NRWT, which is why this test was passing with these baselines.
Mihai Parparita
Comment 4 2010-09-28 00:09:12 PDT
Actually, it seems to be the Mac port that has the precision issues. At the point where the pixel dump is done, the moving box has opacity of 0.5. I would expect the color on the screen to be (0*.5 + 255 * .5, 128 * .5 + 0 * 0.5, 0 * .5 + 0 * 0.5) = (128, 64, 0). However, the Mac port sees (125, 89, 7). Is this color correction in action?
Mihai Parparita
Comment 5 2010-09-28 00:10:39 PDT
Created attachment 69027 [details] Patch
Simon Fraser (smfr)
Comment 6 2010-09-28 11:36:50 PDT
Comment on attachment 69027 [details] Patch We prefer to keep green boxes indicating success. The almost-red box in the result here communicates failure. Maybe it would be better not to use a red background square, but to have a box which surrounds the success location.
Mihai Parparita
Comment 7 2010-09-28 18:11:13 PDT
Created attachment 69147 [details] Patch
Mihai Parparita
Comment 8 2010-09-28 18:12:24 PDT
Actually, I don't have to touch the Mac baselines at all if I make the opacity transition go from 0.7 to 1.0 (like the latest patch). Is that OK?
WebKit Commit Bot
Comment 9 2010-09-28 18:37:57 PDT
Comment on attachment 69147 [details] Patch Clearing flags on attachment: 69147 Committed r68619: <http://trac.webkit.org/changeset/68619>
WebKit Commit Bot
Comment 10 2010-09-28 18:38:02 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.