mozilla-beta: changeset 67228:38fc55b950f945b9d0405d74180a4f4b27f51dad

author	Fabrice Desré <fabrice@mozilla.com>
	Fri, 07 Jan 2011 14:56:56 -0500
changeset 67228	38fc55b950f945b9d0405d74180a4f4b27f51dad
parent 67227	2f30a14583e85f48e76a13defdd59c3cee2ad6ac
child 67229	9c8cc7f65b66b8781ffdf814c8bba6453587010e
push id	1
push user	root
push date	Tue, 26 Apr 2011 22:38:44 +0000
treeherder	mozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder	[talos] [build metrics] [platform microbench] (compared to previous push)
reviewers	mfinkle
bugs	623825

Bug 623825 - Remove 'Options' button for persona themes [r=mfinkle]

--- a/mobile/chrome/content/bindings/extensions.xml
+++ b/mobile/chrome/content/bindings/extensions.xml
@@ -58,16 +58,21 @@
 
           let appDisabled = this.getAttribute("appDisabled");
           if (appDisabled == "true")
             document.getAnonymousElementByAttribute(this, "anonid", "enable-button").setAttribute("disabled", "true");
 
           let isReadOnly = this.getAttribute("isReadonly");
           if (isReadOnly == "true")
             document.getAnonymousElementByAttribute(this, "anonid", "uninstall-button").setAttribute("disabled", "true");
+          
+          // no options for themes
+          // use the unlocalized property, not the typeLabel attribute
+          if (this.addon.type == "theme")
+            document.getAnonymousElementByAttribute(this, "anonid", "options-button").hidden = true;
         ]]>
       </constructor>
 
       <method name="hideOptions">
         <body>
           <![CDATA[
             let box = document.getAnonymousElementByAttribute(this, "anonid", "options-box");
             if (!box.collapsed)