RESOLVED FIXED 63323
Make <video> app cache test exit immediately on failure 
https://bugs.webkit.org/show_bug.cgi?id=63323
Summary Make <video> app cache test exit immediately on failure
Eric Carlson
Reported 2011-06-24 07:50:51 PDT
One of the conditions tested by http/tests/appcache/video.html is setting the video element's src to a JavaScript file included in the manifest to test the handling of a non-media resource from the manifest. This is the last condition tested so the test ends when an 'error' event is be posted, but if the media engine loads the JavaScript file "successfully" the test enters an infinite loop continues until the DRT watchdog timer kills it.
Attachments
Proposed patch. (2.58 KB, patch)
2011-06-24 08:10 PDT, Eric Carlson 		ap: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Eric Carlson
Comment 1 2011-06-24 08:10:12 PDT
Created attachment 98493 [details] Proposed patch.
Alexey Proskuryakov
Comment 2 2011-06-24 08:50:28 PDT
Comment on attachment 98493 [details] Proposed patch. View in context: https://bugs.webkit.org/attachment.cgi?id=98493&action=review > LayoutTests/http/tests/appcache/video.html:46 > + failTest("<br>Loaded file JavaScript file as media!"); Extra "file" here.
Eric Carlson
Comment 3 2011-06-24 09:01:31 PDT
http://trac.webkit.org/changeset/89676
Note You need to log in before you can comment on or make changes to this bug.