UNCONFIRMED 55918
Make LayoutTests robust to latency btwn playing event and frame rendering
https://bugs.webkit.org/show_bug.cgi?id=55918
Summary Make LayoutTests robust to latency btwn playing event and frame rendering
Victoria Kirst
Reported 2011-03-07 19:39:23 PST
Make LayoutTests robust to latency btwn playing event and frame rendering
Attachments
Patch (4.40 KB, patch)
2011-03-07 19:55 PST, Victoria Kirst
no flags
Patch (5.08 KB, patch)
2011-03-09 12:57 PST, Victoria Kirst
no flags
Patch (5.14 KB, patch)
2011-03-09 13:14 PST, Victoria Kirst
commit-queue: commit-queue-
Victoria Kirst
Comment 1 2011-03-07 19:55:08 PST
Victoria Kirst
Comment 2 2011-03-07 19:59:12 PST
These layout tests depended on the play event firing immediately after a call to play(). I added logic to pause and seek so that the pixel results will always be consistent. Rebaselines will come after this patch is committed. See crbug.com/74982
Mihai Parparita
Comment 3 2011-03-07 20:18:52 PST
Comment on attachment 85011 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=85011&action=review > LayoutTests/media/controls-after-reload.html:14 > + video.pause(); Do the pause/currentTime lines have to be inside the "if (window.layoutTestController)" blocks? Generally we want layout tests to behave the same way when run inside the DRT as when they're run inside a normal browser. > LayoutTests/platform/chromium/test_expectations.txt:3059 > +// Temporarily make these tests flaky. Why is this necessary? If you leave them as having just IMAGE expectations, we'll get notified when they start to unexpectedly pass.
Victoria Kirst
Comment 4 2011-03-09 12:57:52 PST
Victoria Kirst
Comment 5 2011-03-09 13:00:17 PST
Gah, sorry about that patch, Mihai! I must have been sleepy when I uploaded it. I have fixed the comments you mentioned (and a few other silly things) and uploaded a new one. I got rid of the GPU modifier on test_expectations.txt because this should affect both software and GPU rendered tests.
Victoria Kirst
Comment 6 2011-03-09 13:14:56 PST
WebKit Commit Bot
Comment 7 2011-03-10 17:57:48 PST
Comment on attachment 85221 [details] Patch Rejecting attachment 85221 [details] from commit-queue. Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=eseidel-cq-sf', 'bu..." exit_code: 2 Last 500 characters of output: c3/JavaArray .... java/lc3/JavaClass .... java/lc3/JavaObject ..................................... java/lc3/StringMethods . java/lc3/forin . java/lc3/instanceof . jquery ........... loader .. mathml ....... mathml/presentation .................. media ................ media/controls-after-reload.html -> failed Exiting early after 1 failures. 13129 tests run. 383.26s total testing time 13128 test cases (99%) succeeded 1 test case (<1%) had incorrect layout 9 test cases (<1%) had stderr output Full output: http://queues.webkit.org/results/8148088
Darin Adler
Comment 8 2011-06-18 12:31:10 PDT
Comment on attachment 85221 [details] Patch Clearing review flag since this patch caused a test failure on the commit bot.
imasaki
Comment 9 2011-10-19 15:18:29 PDT
vrk@, do you have update on this?
Note You need to log in before you can comment on or make changes to this bug.