Closed Bug 455913 Opened 12 years ago Closed 12 years ago

nsIHelperAppLauncher should provide info about content length

Categories

(Core Graveyard :: File Handling, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: romaxa, Assigned: romaxa)

References

Details

Attachments

(2 files)

Attached patch Proposed fixSplinter Review
I think it would be nice to have information about download content size before download starting, especially for mobile using.
Attachment #339286 - Flags: superreview?(cbiesinger)
Attachment #339286 - Flags: review?(cbiesinger)
Summary: nsIExternalHelperAppService should provide info about content length → nsIHelperAppLauncher should provide info about content length
Comment on attachment 339286 [details] [diff] [review]
Proposed fix

+   * The download content length. It might not be available in some cases.

How about:

  * The download content length, or -1 if the length is not available.
Attachment #339286 - Flags: superreview?(cbiesinger)
Attachment #339286 - Flags: superreview+
Attachment #339286 - Flags: review?(cbiesinger)
Attachment #339286 - Flags: review+
Fixed:
http://hg.mozilla.org/mozilla-central/rev/81f3072d75be
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
This needed an iid rev, no?
Assignee: nobody → romaxa
You mean we should change uuid? like this:

-[scriptable, uuid(cc75c21a-0a79-4f68-90e1-563253d0c555)]
+[scriptable, uuid(d9a19faf-497b-408c-b995-777d956b72c0)]
?
Sorry for missing that :/

r+sr=biesi on the IID change
Blocks: 449204
Blocks: 401821
No longer blocks: 449204
Hello,

nsIHelperAppLauncher contentLength doesn't work in Mozilla Thunderbird for mailbox url's.

I always get 11206112 bytes.
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.