Bug 141323

Summary: fast/images/animated-gif-iframe-webkit-transform.html is flaky
Product: WebKit Reporter: Alexey Proskuryakov <ap>
Component: Tools / TestsAssignee: Chris Dumez <cdumez>
Severity: Normal CC: cdumez, commit-queue, dino, koivisto, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

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.