Bug 224339

Summary: [LayoutTests] http/tests/security/resources/reference-movie-cross-origin-allow.py timing out when not passed Content-Length
Product: WebKit Reporter: Chris Gambrell <cgambrell>
Component: Tools / TestsAssignee: Chris Gambrell <cgambrell>
Status: RESOLVED WONTFIX    
Severity: Normal CC: eric.carlson, ews-watchlist, glenn, jbedard, jer.noble, philipj, ryanhaddad, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Result from http/tests/media/reference.mov
none
Patch ews-feeder: commit-queue-

Chris Gambrell
Reported 2021-04-08 13:38:26 PDT
Created attachment 425539 [details] Result from http/tests/media/reference.mov When not specifying the Content-Length header in http/tests/security/resources/reference-movie-cross-origin-allow.py, the test calling the file times out, but does pass when specifying the Content-Length. http/tests/media/resources/reference.mov is what is called by reference-movie-cross-origin-allow.py and its binary content is the body being sent (see attached). http/tests/media/resources/test.mp4 is called in the access_log when specifying the Content-Length header, but is not when omitted (test.mp4 is called from reference.mov). Use run-webkit-tests --root ./build/to/test http/tests/security/video-cross-origin-caching.html to run the file in question and see failure.
Attachments
Result from http/tests/media/reference.mov (434 bytes, text/plain)
2021-04-08 13:38 PDT, Chris Gambrell 		no flags
Details
Patch (2.92 KB, patch)
2021-04-08 13:47 PDT, Chris Gambrell 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-04-08 13:40:20 PDT
<rdar://problem/76419149>
Chris Gambrell
Comment 2 2021-04-08 13:47:05 PDT
Created attachment 425540 [details] Patch
Chris Gambrell
Comment 3 2021-04-15 08:52:15 PDT
This is how CoreMedia is expected to behave and the Content-Length header will need to be specified.
Note You need to log in before you can comment on or make changes to this bug.