Andrea Marchesini [:baku] Assignee
	Description • 7 years ago

By nsIAsyncInputStream contract, ::Available() is allowed to return 0 if the stream is not ready to return data. This is not completely supported by gecko because, in many places, we use ::Available() to know the size of the stream.

For this reason IPCBlobInputStream returns the size of the stream when before starting the retrieving of the remote Stream (before any ::AsyncWait() call). This part should not be needed.

A lot of work has been done in order to fully support nsIAsyncInputStream in gecko: bug 1397627, bug 1397810, bug 1397635 and more. This bug covers the rest: mainly docShell and HTMLFormSubmission.

	Andrea Marchesini [:baku] Assignee
	Comment 5 • 7 years ago

These patches are green on try. Note that I haven't touched nsIWebNavigation::loadURI and ::loadURIWithOptions nor the session history.

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 6 • 7 years ago

Comment on attachment 8906572 [details] [diff] [review]
nsIDocShell::InternalLoad

Please document in the .idl that -1 means unknown length, or "not yet available".

What the XXXbaku mean here? Are you going to fix something there or is there some bug?

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 7 • 7 years ago

Comment on attachment 8906573 [details] [diff] [review]
nsILinkHandler::OnLinkClick

Please document in nsILinkHandler.h what -1 means

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 8 • 7 years ago

Comment on attachment 8906574 [details] [diff] [review]
HTMLFormSubmission

aha, you're removing xxxbaku

Hopefully we have enough tests for all this code.
Could this land after FF57 branching?

	Pulsebot
	Comment 9 • 7 years ago

Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b28ff5ddabbe
nsIDocShell::internalLoad should know the size of the post data inputStream, r=smaug
https://hg.mozilla.org/integration/mozilla-inbound/rev/e77471a16ff7
nsILinkHandler::OnLinkClick and OnLinkClickSync should know the post data inputStream length, r=smaug
https://hg.mozilla.org/integration/mozilla-inbound/rev/90af1cb3c4e3
HTMLFormSubmission should pass the size of the post data inputStream if known, r=smaug
https://hg.mozilla.org/integration/mozilla-inbound/rev/726caa0c36ba
IPCBlobInputStream::Available() should return 0 before any AsyncWait() call, r=smaug

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 10 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/b28ff5ddabbe
https://hg.mozilla.org/mozilla-central/rev/e77471a16ff7
https://hg.mozilla.org/mozilla-central/rev/90af1cb3c4e3
https://hg.mozilla.org/mozilla-central/rev/726caa0c36ba