Bug 37145

Summary:

transitions/transition-end-event-transform.html fails intermittently on Tiger Intel Release bot

Product:

WebKit

Reporter:

Csaba Osztrogonác <ossy>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

REOPENED ---

Severity:

Major

CC:

abarth, ap, commit-queue, dino, pol, tonyg

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Bug Depends on:

Bug Blocks:

33289, 52061

Attachments:

Description	Flags
Patch	none
Patch	none
Failure diff from eseidel-cq-sf	none

Description Csaba Osztrogonác 2010-04-06 04:26:56 PDT

http://build.webkit.org/results/Tiger%20Intel%20Release/r57136%20%2810446%29/transitions/transition-end-event-transform-pretty-diff.html
http://build.webkit.org/results/Tiger%20Intel%20Release/r57141%20%2810449%29/transitions/transition-end-event-transform-pretty-diff.html

Comment 1 Adam Barth 2010-04-06 11:41:17 PDT

Twice in a row:

http://build.webkit.org/results/Tiger%20Intel%20Release/r57133%20(10443)/transitions/transition-end-event-transform-pretty-diff.html

Comment 2 Adam Barth 2010-04-06 13:48:11 PDT

http://build.webkit.org/results/Tiger%20Intel%20Release/r57163%20(10471)/results.html

Comment 3 Adam Barth 2010-04-06 14:10:57 PDT

The real bug is in transition-end-event-set-none.html.  It's leaking an error message into the next test alphabetically.

Comment 4 Adam Barth 2010-04-06 14:27:29 PDT

I have a patch.  Building/testing now.

Comment 5 Adam Barth 2010-04-06 14:36:43 PDT

Created attachment 52669 [details]
Patch

Comment 6 Adam Barth 2010-04-06 14:38:05 PDT

Digging into it more, it turns out the library is already set up to handle these cases, but it's computing the wrong timeout because it doesn't expect two sequential transitions.

+ap in the hopes he'll review my patch because we discussed this test on #webkit a few minutes ago.

Comment 7 Alexey Proskuryakov 2010-04-06 14:44:25 PDT

Comment on attachment 52669 [details]
Patch

r=me

Comment 8 WebKit Commit Bot 2010-04-06 16:27:09 PDT

Comment on attachment 52669 [details]
Patch

Clearing flags on attachment: 52669

Committed r57177: <http://trac.webkit.org/changeset/57177>

Comment 9 WebKit Commit Bot 2010-04-06 16:27:14 PDT

All reviewed patches have been landed.  Closing bug.

Comment 10 Adam Barth 2010-04-07 23:41:47 PDT

Still failing.

Comment 11 Adam Barth 2010-04-07 23:48:20 PDT

Created attachment 52839 [details]
Patch

Comment 12 Eric Seidel (no email) 2010-04-07 23:50:04 PDT

Comment on attachment 52839 [details]
Patch

OK.

Comment 13 Adam Barth 2010-04-07 23:51:32 PDT

Committed r57252.  Hopefully fixed.

Comment 14 Csaba Osztrogonác 2010-04-08 08:59:41 PDT

It fails again: 
http://build.webkit.org/results/Tiger%20Intel%20Release/r57280%20%2810569%29/results.html

Comment 15 Adam Barth 2010-04-08 09:00:44 PDT

Comment on attachment 52839 [details]
Patch

sad face.

Comment 16 Tony Gentilcore 2011-03-14 11:34:36 PDT

*** Bug 51024 has been marked as a duplicate of this bug. ***

Comment 17 WebKit Commit Bot 2011-03-30 00:56:36 PDT

The commit-queue just saw transitions/transition-end-event-transform.html flake (Text diff mismatch) while processing attachment 87290 [details] on bug 57322.
Bot: eseidel-cq-sf  Port: Mac  Platform: Mac OS X 10.6.4

Comment 18 WebKit Commit Bot 2011-03-30 00:56:39 PDT

Created attachment 87479 [details]
Failure diff from eseidel-cq-sf