124868 - i have never seen an Accoun Manager and never hope to see one

Status: VERIFIED FIXED

(MailNews Core :: MIME, defect)

Description

[timeless]

/mailnews/extensions/smime/src/smime-service.js, line 26 --
"@mozilla.org/accounmanager/extension;1?name=smime";

Comment 1

[Boris Zbarsky [:bzbarsky]]

To clarify, the contractid should have a "t" in the "account" part of
"accountmanager"

Comment 2

[timeless]

Attachment: add the t

Comment 3

[Jo Hermans]

We seem to have forgotten this bug ;-) Just setting the flag to remind us next time.

PS : the same is true for /mailnews/extensions/mdn/src/mdn-service.js, line 26

Comment 4

[[e-mail]]

Comment on attachment 68972 [details] [diff] [review]
add the t

Seth's probably a bad choice for reviews at the moment, let's see if someone
else will r/sr it.

Comment 5

[Scott MacGregor]

is anyoone using these contract ID's? Who are we breaking by changing them to
fix a typo?

Comment 6

[timeless]

google doesn't show any users, although it does suggest fixing the spelling error

Comment 7

[timeless]

Comment on attachment 68972 [details] [diff] [review]
add the t

since 1.7 is going to live for a while, we should change this on the branch.
there's no risk. this is just trunk parity.

Comment 8

[Asa Dotzler [:asa]]

Comment on attachment 68972 [details] [diff] [review]
add the t

a=asa (on behalf of drivers) for checkin to 1.7

Comment 9

[timeless]

mozilla/mailnews/extensions/mdn/src/mdn-service.js 	1.5
mozilla/mailnews/extensions/smime/src/smime-service.js 	1.8

Comment 10

[timeless]

mozilla/mailnews/extensions/mdn/src/mdn-service.js 	1.3.26.1
mozilla/mailnews/extensions/smime/src/smime-service.js 	1.6.26.1

Comment 11

[Stephen Donner [:stephend] Not actively reading bugmail]

verified fixed using lxr for the trunk *only*