RESOLVED FIXED 53302
[Windows 7 Release Tests] changesets 76853, 76856, and 76858 broke ~36 animations, compositing, and transitions tests
https://bugs.webkit.org/show_bug.cgi?id=53302
Summary [Windows 7 Release Tests] changesets 76853, 76856, and 76858 broke ~36 animat...
Attachments
Jessie Berlin
Comment 1 2011-01-28 08:23:02 PST
Should I just skip all of these tests? With this many tests failing, that seems to be the wrong thing to do ...
Simon Fraser (smfr)
Comment 2 2011-01-28 08:27:28 PST
Adam should just fix this.
Adam Roben (:aroben)
Comment 3 2011-01-28 10:22:06 PST
(In reply to comment #2) > Adam should just fix this. Yes, but we should skip the tests in the meantime while I work on it.
Jessie Berlin
Comment 4 2011-01-28 11:06:51 PST
Added the failing tests to the skipped list in r76955 http://trac.webkit.org/changeset/76955
Jessie Berlin
Comment 5 2011-01-28 11:27:46 PST
One I did not skip which I now think might also be related to this change is the fast/css/getComputedStyle/getComputedStyle-with-pseudo-element.html test, which also started failing at the same time as the other ~36 tests: http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r76858%20(8728)/fast/css/getComputedStyle/getComputedStyle-with-pseudo-element-pretty-diff.html Since Adam thinks he has a fix for the other tests, I will hold off taking any action on the above test.
WebKit Review Bot
Comment 6 2011-01-28 12:18:03 PST
http://trac.webkit.org/changeset/76955 might have broken Leopard Intel Release (Tests)
Adam Roben (:aroben)
Comment 7 2011-01-28 12:22:06 PST
Fixed in r76968.
Adam Roben (:aroben)
Comment 8 2011-01-28 12:22:39 PST
...so I should close the bug.
WebKit Review Bot
Comment 9 2011-01-28 12:48:14 PST
http://trac.webkit.org/changeset/76968 might have broken Chromium Win Release
Note You need to log in before you can comment on or make changes to this bug.