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>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue, dino, koivisto, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
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.

https://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#showAllRuns=true&revision=179444&showExpectations=true&tests=fast%2Fimages%2Fanimated-gif-iframe-webkit-transform.html
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]
Patch
Comment 4 Alexey Proskuryakov 2015-02-07 15:47:40 PST
Comment on attachment 246222 [details]
Patch

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

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.