1398733 - IPCBlobInputStream::Available() should return 0 if the remote stream has not been received yet

Status: RESOLVED FIXED

(Core :: DOM: File, enhancement)

Description

[Andrea Marchesini [:baku]]

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.

Comment 1

[Andrea Marchesini [:baku]]

Attachment: nsIDocShell::InternalLoad

Comment 2

[Andrea Marchesini [:baku]]

Attachment: nsILinkHandler::OnLinkClick

Comment 3

[Andrea Marchesini [:baku]]

Attachment: HTMLFormSubmission

Comment 4

[Andrea Marchesini [:baku]]

Attachment: IPCBlobInputStream::Available

Comment 5

[Andrea Marchesini [:baku]]

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

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

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?

Comment 7

[Olli Pettay [:smaug][[e-mail]]]

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

Please document in nsILinkHandler.h what -1 means

Comment 8

[Olli Pettay [:smaug][[e-mail]]]

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?

Comment 9

[Pulsebot]

Pushed by [e-mail]:
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

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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