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

Status: UNCONFIRMED

(MailNews Core :: Backend, defect)

Description

[scorpion]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.9) Gecko/20100824 Firefox/3.6.9
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.8) Gecko/20100802 Thunderbird/3.1.2

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

I always get 11206112 bytes.


Reproducible: Always

Comment 1

[Ludovic Hirlimann [:Usul]]

do you have a piece of code that exhibits the bug that you could share ?

Comment 2

[scorpion]

Attachment: Image Number 1

Comment 3

[scorpion]

Attachment: Image Number 2

Comment 4

[scorpion]

Attachment: Image Number 3

Comment 5

[scorpion]

Attachment: dialog.js

Comment 6

[scorpion]

My Add-on:https://addons.mozilla.org/en-US/firefox/addon/181055/

Comment 7

[scorpion]

My installed Extensions:
Show File Size
https://addons.mozilla.org/en-US/firefox/addon/181055/

Attachment Sizes
http://fux.zuhage.de/attachment_sizes/

Comment 8

[scorpion]

Attachment: Image Number 4

Comment 9

[scorpion]

I think dialog.mLauncher.contentLength returns the size of the attachment instead of the selected file.

Comment 10

[Joshua Cranmer [:jcranmer]]

Hmm... as I'm looking through the source code, I am willing to bet that the size is not the size of the attachment but the size of the message as a whole, since it picks up the content length from the message size.

I don't know how the URI load process works in full, but fixing this is likely a lot of trouble (since MIME won't know part size until it starts getting the message stream, i.e., after the dialog has been clicked through [ I think ], so this information would need to be cached somewhere from the first MIME run... if there is one.

Comment 11

[scorpion]

// https://developer.mozilla.org/en/Extensions/Thunderbird/HowTos/Common_Thunderbird_Use_Cases/View_Message#Get_Message_Header_by_URI.2fURL
this:
dialog.mLauncher.source.QueryInterface(Components.interfaces.nsIMsgMessageUrl).messageHeader.messageSize;
is the same as: dialog.mLauncher.contentLength;

// https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsIMsgDBHdr
Can you add a new attribute like messageSize to get the file size from the attachment (e.g. attachmentSize)? Would that be possible? 

Then i can use this: dialog.mLauncher.source.QueryInterface(Components.interfaces.nsIMsgMessageUrl).messageHeader.attachmentSize;

Comment 12

[scorpion]

Attachment: dialog2.js

Comment 13

[David :Bienvenu]

iirc, gloda is now storing attachment sizes but no one else does.

Comment 14

[scorpion]

what does gloda mean?

Comment 15

[Ludovic Hirlimann [:Usul]]

Removing myslef on all the bugs I'm cced on. Please NI me if you need something on MailNews Core bugs from me.