482296 - Add attribute to nsIMsgMailNewsUrl to determine whether this is for a message

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[David Humphrey (:humph)]

Spin-off from bug 467159 comment c3: it would be nice to solve this for all callers who might have a -message url (e.g., imap-message://).  After discussions on irc, the suggestion is to add a readonly attribute named isMessageUri that returns true in the case of a message uri.

Comment 1

[David Humphrey (:humph)]

Attachment: Patch to add isMessgeUri attribute

I have a new patch for bug 467159 based on this that I've tested, and it works.  Hopefully I've understood our discussion and done what you wanted.

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 366472 [details] [diff] [review]
Patch to add isMessgeUri attribute

Please use doxygen-style comments in IDL whenever possible, even if it doesn't match local style, because that allows automated doc extraction tools to pick it up.  sr=dmose with that change.

Comment 3

[David :Bienvenu]

Comment on attachment 366472 [details] [diff] [review]
Patch to add isMessgeUri attribute

I think that means using /// instead of /.

I haven't tested this, because there are no callers yet, but it looks good.

Comment 4

[David Humphrey (:humph)]

Attachment: Patch v2 with doxygen style comment

I've updated the comment style to use /// and also modified the text slightly to match other IDL doxygen comments I found in the tree.

Comment 5

[Mark Banner (:standard8)]

Patch checked in: http://hg.mozilla.org/comm-central/rev/e61d6cba56a1