Bug 28275

Summary: media/video-played.html always times out on Windows
Product: WebKit Reporter: Adam Roben (:aroben) <aroben>
Component: Tools / TestsAssignee: Eric Carlson <eric.carlson>
Status: RESOLVED FIXED    
Severity: Normal CC: eric.carlson
Priority: P2 Keywords: InRadar, LayoutTestFailure, PlatformOnly
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
URL: http://build.webkit.org/results/Windows%20Debug%20(Tests)/r47214%20(3565)/media/video-played-pretty-diff.html
Attachments:
Description Flags
proposed patch simon.fraser: review+

Description Adam Roben (:aroben) 2009-08-13 13:16:27 PDT
To reproduce:

1. run-webkit-tests --debug media/video-played.html

The test always times out in Debug builds. Maybe it needs to be split up to make it take less time?
Comment 1 Adam Roben (:aroben) 2009-08-13 13:19:46 PDT
Looks like it times out in Release builds, too: http://build.webkit.org/results/Windows%20Release%20(Tests)/r47216%20(3350)/media/video-played-pretty-diff.html
Comment 2 Adam Roben (:aroben) 2009-08-13 13:20:12 PDT
<rdar://problem/7141132>
Comment 3 Eric Carlson 2009-08-14 09:56:26 PDT
Created attachment 34850 [details]
proposed patch
Comment 4 Simon Fraser (smfr) 2009-08-14 10:49:57 PDT
Comment on attachment 34850 [details]
proposed patch

> +//@@@@@ Uncomment the following line to log the time each "video-played" sub-test takes in test output
> +//@@@@@ logTestTiming = true;

Weird comment style.
Comment 5 Eric Carlson 2009-08-14 11:03:35 PDT
>> +//@@@@@ Uncomment the following line to log the time each "video-played" sub-test takes in test output
>> +//@@@@@ logTestTiming = true;

>Weird comment style.

Yes, I did that purposely so it stands out visually for someone reading it later. Understanding a failure in one of these tests is almost impossible without extra the logging so I wanted to make this debugging tool obvious.
Comment 6 Eric Carlson 2009-08-14 11:05:32 PDT
http://trac.webkit.org/changeset/47287