690778 - Update nsFeedSniffer.cpp (Port part of Bug 589292 Add contentDisposition{Filename} properties to nsIChannel).

Status: RESOLVED FIXED

(SeaMonkey :: Feed Discovery and Preview, defect)

Description

[Philip Chee]

q.v. Core Bug 589292 Add contentDisposition{Filename} properties to nsIChannel.

Comment 1

[Philip Chee]

Attachment: Patch v1.0 straight port.

Well it compiles and I don't get any errors when I run SeaMonkey but I'm not sure how to test this.

Comment 2

[[e-mail]]

Comment on attachment 563743 [details] [diff] [review]
Patch v1.0 straight port.

>+  if (NS_SUCCEEDED(rv) && disp == nsIChannel::DISPOSITION_ATTACHMENT)
>+    return PR_TRUE;
> 
>   return PR_FALSE;
This can of course be improved upon. In fact, the whole function could be written in two lines of code should you so wish, although you would wrap them to four lines so it wouldn't really represent a saving.

Comment 3

[[e-mail]]

(In reply to Philip Chee from comment #1)
> I'm not sure how to test this.
You would need to have a server so that you could configure it to send either:
a) a feed with the correct content type (which just displays without sniffing)
b) a feed with a commonly misconfigured content type (which would get sniffed)
c) as b) but with content-disposition: attachment (which would get downloaded)

Comment 4

[Philip Chee]

> This can of course be improved upon. In fact, the whole function could be written in
> two lines of code should you so wish, although you would wrap them to four lines so it
> wouldn't really represent a saving.
Hmm. Well if it ain't broke...

Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/3f0cb19942bc