WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
141323
fast/images/animated-gif-iframe-webkit-transform.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=141323
Summary
fast/images/animated-gif-iframe-webkit-transform.html is flaky
Alexey Proskuryakov
Reported
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
Attachments
Patch
(2.99 KB, patch)
2015-02-07 15:45 PST
,
Chris Dumez
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1
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.
Chris Dumez
Comment 2
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().
Chris Dumez
Comment 3
2015-02-07 15:45:45 PST
Created
attachment 246222
[details]
Patch
Alexey Proskuryakov
Comment 4
2015-02-07 15:47:40 PST
Comment on
attachment 246222
[details]
Patch rs=me
WebKit Commit Bot
Comment 5
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
>
WebKit Commit Bot
Comment 6
2015-02-07 16:33:06 PST
All reviewed patches have been landed. Closing bug.
Alexey Proskuryakov
Comment 7
2015-02-09 23:29:09 PST
Thanks, this fixed it.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug