As mentioned in bug13596, we need more test case about onprogress event handler. On the top of my head: - addEventListener("progress", mycallback, XXX) should work - add checks for both interfaces (LSProgressEvent and ProgressEvent) - calling abort() should reset m_receivedLength - ...
Created attachment 21444 [details] Add 5 test cases
Comment on attachment 21444 [details] Add 5 test cases Nice! I love test cases!
Committed in r34320.
Comment on attachment 21444 [details] Add 5 test cases View in context: https://bugs.webkit.org/attachment.cgi?id=21444&action=review > LayoutTests/http/tests/xmlhttprequest/xmlhttprequest-no-content-length-onProgress.html:35 > +// If content length is not given, it should not be called > +req.onprogress = onProgressFailed; > +req.open("GET", "resources/noContentLength.asis", false); > +req.send(null); This test seems wrong, there is no reason to not fire the progress event for responses with chunked encoding. I'll fix. > LayoutTests/http/tests/xmlhttprequest/resources/noContentLength.asis:1 > +<html> This is not a properly formatted .asis file. It makes Apache return an empty response with "Content-Length: 0".