Bug 53302

Summary: [Windows 7 Release Tests] changesets 76853, 76856, and 76858 broke ~36 animations, compositing, and transitions tests
Product: WebKit Reporter: Jessie Berlin <jberlin>
Component: Layout and RenderingAssignee: Adam Roben (:aroben) <aroben>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, aroben, cmarrin, eric, jberlin, simon.fraser, webkit.review.bot
Priority: P2 Keywords: LayoutTestFailure, PlatformOnly, Regression
Version: 528+ (Nightly build)   
Hardware: All   
OS: Windows 7   

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.