59558 - cannot uninstall skins via component as spec'd

Status: VERIFIED FIXED

(Core Graveyard :: Skinability, defect, P3)

Description

[andreww]

in nsiChromeRegistry.idl there is a method which is supposed to allow you to 
uninstall a theme:

http://lxr.mozilla.org/seamonkey/source/rdf/chrome/public/
nsIChromeRegistry.idl#86

I modified the pref-themes.xul :
http://lxr.mozilla.org/seamonkey/source/xpfe/components/prefwindow/resources/﷒0﷓

to add a button which would call the uninstallskin method, using the same 
argument that "selectskin" uses.

Instead of the skin being removed, I get the following error:
uncaught exception: [Exception... "Component returned failure code:
0x800004003 (NS_ERROR_INVALID_POINTER) [nsIChromeRegistry.uninstallSkin]" 
nsresult: "0x80004003 (NS_ERROR_INVALID_POINTER)" location: "JS frame :: 

etc. etc.....

Comment 1

[andreww]

changing owner...

Comment 2

[Peter Trudelle]

->moz0.9

Comment 3

[hangas]

changing component to skinability

Comment 4

[David Hyatt]

Attachment: Patch to enable uninstallation of a skin

Comment 5

[David Hyatt]

Ready for r and sr.

Comment 6

[Chris Waterson]

[s]r=waterson. nice, you sexy RDF bastard.

Comment 7

[David Hyatt]

Fixed.

Comment 8

[Patty Mac]

Marking verified on Mac.

Comment 9

[Peter Lairo]

Is this really fixed? I don't see an uninstall button under Themes - Prefs!

Comment 10

[:deanis74]

Peter, this looks to be the back-end work.  I filed a bug about adding something
to the prefs panel as bug 71194.

Comment 11

[David Hyatt]

The API now works.  This bug is not covering the UI aspect.