Bug 487250 - 'empty' attribute for textboxes with emptytext conflicts with xul templates. r=enn

browser/base/content/browser.css

@@ -38,7 +38,7 @@ toolbar[printpreview="true"] {
 
 #urlbar[pageproxystate="invalid"] > #urlbar-icons > :not(#go-button) ,
 #urlbar[pageproxystate="valid"] > #urlbar-icons > #go-button ,
-#urlbar[empty="true"] > #urlbar-icons > #go-button {
+#urlbar[isempty="true"] > #urlbar-icons > #go-button {
   visibility: collapse;
 }
 

browser/themes/pinstripe/browser/browser.css

@@ -1352,7 +1352,7 @@ richlistitem[selected="true"][current="true"] > hbox > .ac-result-type-bookmark,
   color: inherit;
 }
 
-#editBMPanel_tagsField[empty="true"] {
+#editBMPanel_tagsField[isempty="true"] {
   color: #bbb !important;
 }
 

browser/themes/pinstripe/browser/searchbar.css

@@ -138,10 +138,6 @@
   background-image: url("chrome://browser/skin/urlbar/endcap-focused-graphite-rtl.png");
 }
 
-#searchbar[empty="true"] .search-go-button {
-  visibility: hidden;
-}
-
 .search-go-button {
   padding: 1px;
   list-style-image: url("chrome://browser/skin/Search.png");

toolkit/content/tests/widgets/test_textbox_emptytext.xul

@@ -29,7 +29,7 @@ function doTests() {
   var t1 = $("t1");
 
   t1.emptyText = 1;
-  ok(t1.hasAttribute("empty"),  "emptyText but no value => 'empty' attribute is present");
+  ok(t1.hasAttribute("isempty"), "emptyText but no value => 'isempty' attribute is present");
   ok("1" === t1.label,          "emptyText exposed as label");
   ok(""  === t1.value,          "emptyText not exposed as value");
 
@@ -38,7 +38,7 @@ function doTests() {
   ok("1" === t1.emptyText,      "emptyText persists after setting label");
 
   t1.value = 3;
-  ok(!t1.hasAttribute("empty"), "value present => 'empty' attribute not present");
+  ok(!t1.hasAttribute("isempty"), "value present => 'isempty' attribute not present");
   ok("3" === t1.value,          "value setter/getter works while emptyText is present");
   ok("1" === t1.emptyText,      "emptyText persists after setting value");
 

toolkit/content/textbox.css

@@ -12,20 +12,20 @@ html|*.textbox-textarea {
   text-shadow: inherit;
 }
 
-textbox[empty="true"] html|*.textbox-input ,
-textbox[empty="true"] html|*.textbox-textarea {
+textbox[isempty="true"] html|*.textbox-input ,
+textbox[isempty="true"] html|*.textbox-textarea {
   text-align: left;
   direction: ltr;
 }
 
-textbox[empty="true"][chromedir="rtl"] html|*.textbox-input ,
-textbox[empty="true"][chromedir="rtl"] html|*.textbox-textarea {
+textbox[isempty="true"][chromedir="rtl"] html|*.textbox-input ,
+textbox[isempty="true"][chromedir="rtl"] html|*.textbox-textarea {
   text-align: right;
   direction: rtl;
 }
 
-textbox[empty="true"] html|*.textbox-input[emptytextdelay="true"] ,
-textbox[empty="true"] html|*.textbox-textarea[emptytextdelay="true"] {
+textbox[isempty="true"] html|*.textbox-input[emptytextdelay="true"] ,
+textbox[isempty="true"] html|*.textbox-textarea[emptytextdelay="true"] {
   color: transparent !important;
 }
 

toolkit/content/widgets/textbox.xml

@@ -55,7 +55,7 @@
       </property>
 
       <property name="value"
-                onget="return this.hasAttribute('empty') ? '' : this.inputField.value;">
+                onget="return this.hasAttribute('isempty') ? '' : this.inputField.value;">
         <setter><![CDATA[
           if (val) {
             // clear the emptyText _before_ setting a new non-empty value
@@ -129,7 +129,7 @@
 
       <property name="controllers"    readonly="true" onget="return this.inputField.controllers"/>
       <property name="textLength"     readonly="true"
-                                      onget="return this.hasAttribute('empty') ?
+                                      onget="return this.hasAttribute('isempty') ?
                                                     0 : this.inputField.textLength;"/>
       <property name="selectionStart" onset="this.inputField.selectionStart = val; return val;"
                                       onget="return this.inputField.selectionStart;"/>
@@ -168,8 +168,8 @@
               !this.value &&
               this.emptyText) {
 
-            if (!this.hasAttribute("empty")) {
-              this.setAttribute("empty", "true");
+            if (!this.hasAttribute("isempty")) {
+              this.setAttribute("isempty", "true");
 
               // Hide the emptytext for a bit, in case the textbox will be focused subsequently
               this.inputField.setAttribute("emptytextdelay", "true");
@@ -189,7 +189,7 @@
 
       <method name="_clearEmptyText">
         <body><![CDATA[
-          if (this.hasAttribute("empty")) {
+          if (this.hasAttribute("isempty")) {
             this.inputField.value = "";
             try {
               var transactionManager = this.editor.transactionManager;
@@ -200,7 +200,7 @@
                   transactionManager.numberOfRedoItems == 0)
                 transactionManager.clear();
             } catch (e) {}
-            this.removeAttribute("empty");
+            this.removeAttribute("isempty");
           }
         ]]></body>
       </method>
@@ -323,7 +323,7 @@
                 onset="this.setAttribute('timeout', val); return val;"
                 onget="return parseInt(this.getAttribute('timeout')) || 0;"/>
       <property name="value"
-                onget="return this.hasAttribute('empty') ? '' : this.inputField.value;">
+                onget="return this.hasAttribute('isempty') ? '' : this.inputField.value;">
         <setter><![CDATA[
           if (val) {
             // clear the emptyText _before_ setting a new non-empty value
@@ -408,7 +408,7 @@
         ]]></setter>
       </property>
       <property name="value"
-                onget="return this.hasAttribute('empty') ? '' : this.inputField.value;">
+                onget="return this.hasAttribute('isempty') ? '' : this.inputField.value;">
         <setter><![CDATA[
           if (val) {
             // clear the emptyText _before_ setting a new non-empty value

toolkit/themes/gnomestripe/global/textbox.css

@@ -60,7 +60,7 @@ textbox {
   color: -moz-FieldText;
 }
 
-textbox[empty="true"] {
+textbox[isempty="true"] {
   color: GrayText;
 }
 

toolkit/themes/pinstripe/global/textbox.css

@@ -64,7 +64,7 @@ textbox {
   color: -moz-FieldText;
 }
 
-textbox[empty="true"] {
+textbox[isempty="true"] {
   color: GrayText;
 }
 

toolkit/themes/winstripe/global/textbox-aero.css

@@ -1,5 +1,5 @@
 %include textbox.css
 
-textbox[empty="true"]:-moz-system-metric(windows-default-theme) {
+textbox[isempty="true"]:-moz-system-metric(windows-default-theme) {
   font-style: italic;
 }

toolkit/themes/winstripe/global/textbox.css

@@ -60,7 +60,7 @@ textbox {
   color: -moz-FieldText;
 }
 
-textbox[empty="true"] {
+textbox[isempty="true"] {
   color: GrayText;
 }