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+

Adam Roben (:aroben)
Reported 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?
Attachments
proposed patch (27.14 KB, patch)
2009-08-14 09:56 PDT, Eric Carlson
simon.fraser: review+
Adam Roben (:aroben)
Comment 1 2009-08-13 13:19:46 PDT
Adam Roben (:aroben)
Comment 2 2009-08-13 13:20:12 PDT
Eric Carlson
Comment 3 2009-08-14 09:56:26 PDT
Created attachment 34850 [details] proposed patch
Simon Fraser (smfr)
Comment 4 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.
Eric Carlson
Comment 5 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.
Eric Carlson
Comment 6 2009-08-14 11:05:32 PDT
Note You need to log in before you can comment on or make changes to this bug.