Incorrect file loaded reporting

[mkszepp]

When the upload is completed, request.onprogress will be fired at the same time like request.onloadend.
As the upload is completed with request.onprogress the server returns the Content-Length and the browser takes this size... In my case the returned Content-Length is lower, because i don't return the image and so the progress bar is going back.

My debug console.logs:

To fix this behavier we should replace the file.loaded set in request.onprogress to:
file.loaded = Math.max(evt.loaded, file.loaded); instead of file.loaded = evt.loaded;

Comment copied from: #1002 (comment)

[mkszepp]

@gilest one think, that we should discuss.. should we check in the "onprogress" if file.loaded is not > than file.size, otherwise it can occures that a file has more than 100%

[mkszepp]

@gilest friendly ping

[gilest]

Hey @mkszepp thanks for this! As before I think this looks more correct than the previous implementation.

It would be cool to take your collected logs and use them to build a regression test but I don't think that should block this being released.

[gilest]

Released in 8.3.0