Bug 60055 - media/video-volume-slider.html causes GTK 64 bit debug bot to time out
Summary: media/video-volume-slider.html causes GTK 64 bit debug bot to time out
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 28220 60693
Blocks:
  Show dependency tree / graph
 
Reported: 2011-05-03 12:44 PDT by Mario Sanchez Prada
Modified: 2011-05-16 00:13 PDT (History)
6 users (show)

See Also:

Attachments
proposed patch (2.25 KB, patch)
2011-05-06 01:33 PDT, Philippe Normand 		no flags Details | Formatted Diff | Diff
updated patch (2.04 KB, patch)
2011-05-12 02:00 PDT, Philippe Normand 		mrobinson: review+
webkit.review.bot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ec2-cr-linux-01 (1.21 MB, application/zip)
2011-05-12 02:16 PDT, WebKit Review Bot 		no flags Details
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mario Sanchez Prada 2011-05-03 12:44:35 PDT 
It was observed that, after revision r85586 [1], the media tests in the 64 bit started to time out always in the same place.

After checking the tests_run.txt file [2], it looks like the problem is in this test, so I'm skipping it now to unblock this bot while we don't find a better solution for it.

[1] http://trac.webkit.org/changeset/85586
[2] http://build.webkit.org/results/GTK%20Linux%2064-bit%20Debug/r85620%20%2822092%29/tests_run.txt
Comment 1 Philippe Normand 2011-05-04 00:20:56 PDT 
It could be the event sender not hiting the correct position in the page
Comment 2 Philippe Normand 2011-05-06 01:33:24 PDT 
Created attachment 92563 [details]
proposed patch
Comment 3 Philippe Normand 2011-05-12 02:00:48 PDT 
Created attachment 93269 [details]
updated patch
Comment 4 WebKit Review Bot 2011-05-12 02:15:59 PDT 
Comment on attachment 93269 [details]
updated patch

Attachment 93269 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/8688667

New failing tests:
media/video-volume-slider.html
Comment 5 WebKit Review Bot 2011-05-12 02:16:04 PDT 
Created attachment 93270 [details]
Archive of layout-test-results from ec2-cr-linux-01

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-01  Port: Chromium  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Comment 6 Philippe Normand 2011-05-12 02:17:54 PDT 
(In reply to comment #5)
> Created an attachment (id=93270) [details]
> Archive of layout-test-results from ec2-cr-linux-01
> 
> The attached test failures were seen while running run-webkit-tests on the chromium-ews.
> Bot: ec2-cr-linux-01  Port: Chromium  Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick

See bug 60293.
Would be nice if the EWS could handle bug dependencies :)
Comment 7 Philippe Normand 2011-05-12 02:18:17 PDT 
Bug 60693 I mean!
Comment 8 Martin Robinson 2011-05-12 09:01:48 PDT 
Comment on attachment 93269 [details]
updated patch

View in context: https://bugs.webkit.org/attachment.cgi?id=93269&action=review

> LayoutTests/media/video-volume-slider.html:30
> +                x = muteCoords[0];
> +                y = muteCoords[1];
>                  eventSender.mouseMoveTo(x, y);

You could just pass the coordinates directly here ala eventSender.moveMouseTo(muteCoords[0], museCoords[1]); without much loss of explication.
Comment 9 Philippe Normand 2011-05-16 00:13:15 PDT 
Committed r86535: <http://trac.webkit.org/changeset/86535>