Bug 141323 - fast/images/animated-gif-iframe-webkit-transform.html is flaky
Summary: fast/images/animated-gif-iframe-webkit-transform.html is flaky
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
Depends on:
Reported: 2015-02-05 20:29 PST by Alexey Proskuryakov
Modified: 2015-02-09 23:29 PST (History)
5 users (show)

See Also:

Patch (2.99 KB, patch)
2015-02-07 15:45 PST, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2015-02-05 20:29:48 PST
fast/images/animated-gif-iframe-webkit-transform.html is flaky - mostly on debug bots, but also on release sometimes.

-PASS isFirstImagePaused() is false
+FAIL isFirstImagePaused() should be false. Was true.

Comment 1 Alexey Proskuryakov 2015-02-05 20:46:11 PST
This is a fairly new test. Chris, do you know what could be wrong here?

I couldn't reproduce locally, even with many parallel processes.
Comment 2 Chris Dumez 2015-02-05 20:51:06 PST
I wrote this test. I'll take a look tomorrow.

Like other gif animations test relying on internals.hasPausedImageAnimations(), it doesn't work (reliably) on WK1. We already have a section in wk1 TestsExpectations to skip those.

However, on WK2 it should work so I'll have to investigate. My initial feeling is that we'll like need to use shouldBecomeEqual() for isFirstImagePaused(), like we already do for isSecondImagePaused().
Comment 3 Chris Dumez 2015-02-07 15:45:45 PST
Created attachment 246222 [details]
Comment 4 Alexey Proskuryakov 2015-02-07 15:47:40 PST
Comment on attachment 246222 [details]

Comment 5 WebKit Commit Bot 2015-02-07 16:32:59 PST
Comment on attachment 246222 [details]

Clearing flags on attachment: 246222

Committed r179787: <http://trac.webkit.org/changeset/179787>
Comment 6 WebKit Commit Bot 2015-02-07 16:33:06 PST
All reviewed patches have been landed.  Closing bug.
Comment 7 Alexey Proskuryakov 2015-02-09 23:29:09 PST
Thanks, this fixed it.