166795 - "mozMapi" should be "MozillaMapi"

Status: VERIFIED FIXED

(MailNews Core :: Simple MAPI, defect)

Description

[Henrik Gemal]

current we have this in the reg.db:
HKEY_CLASSES_ROOT\mozMapi
HKEY_CLASSES_ROOT\mozMapi.1

but all the other mozilla entries are:
HKEY_CLASSES_ROOT\MozillaXHTML
HKEY_CLASSES_ROOT\MozillaXBM

So:
HKEY_CLASSES_ROOT\mozMapi
should be:
HKEY_CLASSES_ROOT\MozillaMapi
to be consistent!

code should be changed here:
http://lxr.mozilla.org/mozilla/source/mailnews/mapi/mapihook/src/nsMapiRegistryU
tils.cpp#573

20020904

Comment 1

[Henrik Gemal]

Attachment: mozMapi -> MozillaMapi to keep naming

Comment 2

[timeless]

Comment on attachment 101532 [details] [diff] [review]
mozMapi -> MozillaMapi to keep naming

<gemal:#mozilla> it's bug 125877 and bug 129088 and bug 169371. We could add to
bug 129088 that it should also remove mozMapi entries

please :)

Comment 3

[(not reading, please use [e-mail] instead)]

please let rdayal do a review.

if it looks good to him, then sr=sspitzer

Comment 4

[Rajiv Dayal]

looks fine, r=rdayal.

Comment 5

[Henrik Gemal]

seth or rajiv: could you check it in since I dont have cvs access

Comment 6

[[e-mail] (gone - use [e-mail] instead)]

Henrik, you're going to have to mail for a= if you want inclusion into the
Mozilla1.2 trunk.

Comment 7

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 101532 [details] [diff] [review]
mozMapi -> MozillaMapi to keep naming

marking sr=sspitzer

a=[e-mail]

Comment 8

[Henrik Gemal]

stephen: could you do the checkin?

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

Nope, sorry, I've given up checking in and bug fixing.  Purely QA now.

Comment 10

[Henrik Gemal]

anyone else wanna do the checkin?

Comment 11

[Boris Zbarsky [:bzbarsky]]

checked in to the trunk

Comment 12

[Rajiv Dayal]

thanks Boris for landing this. Marking this as fixed.

Comment 13

[[e-mail] (gone - use [e-mail] instead)]

Verified FIXED through LXR and my own code tree.

1.10 bzbarsky%mit.edu Oct 15 23:16   Better naming for mapi registry keys. Bug
166795, patch by
[e-mail] (Henrik Gemal), r=rdayal, sr=sspitzer, a=rjesup