Bug 68841 We don't underline accesskeys for XUL checkboxes or radiobuttons

Bug 260657 Txul, Ts regression caused by checkbox/radio underlines
Relanding relevant parts of patches following aviary branch landing

toolkit/content/widgets/general.xml

@@ -18,8 +18,27 @@
                                   onget="return this.getAttribute('disabled') == 'true';"/>
       <property name="image"      onset="return this.setAttribute('image',val);"
                                   onget="return this.getAttribute('image');"/>
-      <property name="accessKey"  onset="return this.setAttribute('accesskey',val);"
-                                  onget="return this.getAttribute('accesskey');"/>
+      <property name="accessKey">
+        <getter>
+          <![CDATA[
+            return this.labelElement ? this.labelElement.accessKey : this.getAttribute('accesskey');
+          ]]>
+        </getter>
+        <setter>
+          <![CDATA[
+            // If there is a label, store the accesskey on the labelElement
+            if (this.labelElement) {
+              this.labelElement.accessKey = val;
+            }
+            else {
+              this.setAttribute('accesskey', val);
+            }
+            return val;
+          ]]>
+        </setter>
+      </property>
+      
+      <field name="labelElement"/>
     </implementation>      
   </binding>
 

toolkit/content/widgets/radio.xml

@@ -358,7 +358,7 @@
       </xul:hbox>
     </content>
 
-    <implementation implements="nsIDOMXULSelectControlItemElement, nsIAccessibleProvider">
+    <implementation implements="nsIDOMXULSelectControlItemElement, nsIDOMXULLabeledControlElement, nsIAccessibleProvider">
       <constructor>
         <![CDATA[
           // Just clear out the parent's cached list of radio children

toolkit/content/widgets/text.xml

@@ -2,8 +2,9 @@
 
 <bindings id="textBindings"
    xmlns="http://www.mozilla.org/xbl"
-   xmlns:xul="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
-  
+   xmlns:xul="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
+   xmlns:html="http://www.w3.org/1999/xhtml">
+
   <!-- bound to <description>s -->
   <binding id="text-base">
     <implementation implements="nsIDOMXULDescriptionElement, nsIAccessibleProvider">
@@ -17,7 +18,7 @@
       </property>
       <property name="disabled" onget="return this.hasAttribute('disabled');"
                                 onset="if (val) this.setAttribute('disabled', 'true');
-                                       else this.removeAttribute('disabled'); 
+                                       else this.removeAttribute('disabled');
                                        return val;"/>
       <property name="value" onget="return this.getAttribute('value');"
                              onset="this.setAttribute('value', val); return val;"/>
@@ -28,14 +29,158 @@
 
   <binding id="text-label" extends="chrome://global/content/bindings/text.xml#text-base">
     <implementation implements="nsIDOMXULLabelElement">
-      <property name="accessKey" onget="return this.getAttribute('accesskey');"
-                                 onset="this.setAttribute('accesskey', val); return val;"/>
-      <property name="control" onget="return this.getAttribute('control');"
-                               onset="this.setAttribute('control', val); return val;"/>
+      <property name="accessKey">
+        <getter>
+          <![CDATA[
+            var accessKey = this.getAttribute('accesskey');
+            return accessKey ? accessKey[0] : null;
+          ]]>
+        </getter>
+        <setter>
+          <![CDATA[
+            this.setAttribute('accesskey', val);
+            return val;
+          ]]>
+        </setter>
+      </property>
+
+      <property name="control" onget="return getAttribute('control');">
+        <setter>
+          <![CDATA[
+            // After this gets set, the label will use the binding #label-control
+            this.setAttribute('control', val);
+            return val;
+          ]]>
+        </setter>
+      </property>
     </implementation>
   </binding>
 
   <binding id="label-control" extends="chrome://global/content/bindings/text.xml#text-label">
+    <content>
+      <html:span anonid="accessKeyParens"><children/></html:span>
+    </content>
+    <implementation implements="nsIDOMXULLabelElement">
+      <constructor>
+        <![CDATA[
+          this.formatAccessKey();
+        ]]>
+      </constructor>
+
+      <method name="formatAccessKey">
+        <body>
+          <![CDATA[
+            var control = this.labeledControlElement;
+            if (!control) {
+              var bindingParent = document.getBindingParent(this);
+              if (bindingParent instanceof Components.interfaces.nsIDOMXULLabeledControlElement) {
+                control = bindingParent; // For controls that make the <label> an anon child
+              }
+            }
+            if (control) {
+              control.labelElement = this;
+            }
+
+            var afterLabel = document.getAnonymousElementByAttribute(this, "anonid", "accessKeyParens");
+            afterLabel.textContent = ""; // This does not clear real nodes!
+
+            var oldAccessKey = this.getElementsByAttribute('class', 'accesskey').item(0);
+            if (oldAccessKey) { // Clear old accesskey
+              if (oldAccessKey.previousSibling instanceof Text) {
+                oldAccessKey.previousSibling.appendData(oldAccessKey.textContent)
+              }
+              else {
+                oldAccessKey.parentNode.insertBefore(oldAccessKey.firstChild, oldAccessKey);
+              }
+              oldAccessKey.parentNode.removeChild(oldAccessKey);
+            }
+
+            var accessKey = this.accessKey;
+            var labelText = this.textContent;
+            if (!accessKey || !labelText || !control) {
+              return;
+            }
+            var accessKeyIndex = labelText.indexOf(accessKey);
+            if (accessKeyIndex < 0) { // Try again in upper case
+              accessKeyIndex = labelText.toUpperCase().indexOf(accessKey.toUpperCase());
+            }
+            var span = document.createElementNS("http://www.w3.org/1999/xhtml", "span");
+            span.className = "accesskey";
+            if (accessKeyIndex < 0) {
+              // If accesskey is not in string, append in parentheses
+              afterLabel.textContent = " (";
+              span.textContent = accessKey.toUpperCase();
+              afterLabel.appendChild(span);
+              afterLabel.appendChild(document.createTextNode(")"));
+              return;
+            }
+            var treeWalker = document.createTreeWalker(this, NodeFilter.SHOW_TEXT, null, true);
+            var accessKeyNode = treeWalker.nextNode();
+            
+            while (accessKeyIndex >= accessKeyNode.length) {
+              accessKeyIndex -= accessKeyNode.length;
+              accessKeyNode = treeWalker.nextNode();
+            }
+
+            // Now wrap the access key in a <span class="accesskey"/>
+            if (accessKeyIndex) {
+              accessKeyNode = accessKeyNode.splitText(accessKeyIndex); // returns 2nd node from split
+            }
+            accessKeyNode.parentNode.insertBefore(span, accessKeyNode);
+            if (accessKeyNode.length > 1) {
+              accessKeyNode.splitText(1);
+            }
+            span.appendChild(accessKeyNode);
+          ]]>
+        </body>
+      </method>
+
+      <property name="accessKey">
+        <getter>
+          <![CDATA[
+            var accessKey = this.getAttribute('accesskey');
+            if (!accessKey) {
+              var labeledEl = this.labeledControlElement;
+              if (labeledEl) {
+                accessKey = labeledEl.getAttribute('accesskey');
+              }
+            }
+            return accessKey ? accessKey[0] : null;
+          ]]>
+        </getter>
+        <setter>
+          <![CDATA[
+            // If the control already has an accesskey store it there as well
+            var control = this.labeledControlElement;
+            if (control && control.hasAttribute('accesskey')) {
+              control.setAttribute('accesskey', val);
+            }
+            this.setAttribute('accesskey', val);
+            this.formatAccessKey();
+            return val;
+          ]]>
+        </setter>
+      </property>
+
+      <property name="labeledControlElement" readonly="true"
+                onget="var control = this.control; return control ? document.getElementById(control) : null;" />
+
+      <property name="control" onget="return this.getAttribute('control');">
+        <setter>
+          <![CDATA[
+            var control = this.labeledControlElement;
+            if (control) {
+              control.labelElement = null; // No longer pointed to be this label
+            }
+            this.setAttribute('control', val);
+            this.formatAccessKey();
+            return val;
+          ]]>
+        </setter>
+      </property>
+
+    </implementation>
+
     <handlers>
       <handler event="click" action="if (this.disabled) return;
                                      var controlElementID = this.getAttribute('control');
@@ -46,7 +191,7 @@
                                        controlElement.focus();
                                     "/>
     </handlers>
-  </binding>  
+  </binding>
 
   <binding id="text-link" extends="chrome://global/content/bindings/text.xml#text-label">
     <handlers>

toolkit/content/xul.css

@@ -624,10 +624,14 @@ label.text-link {
   -moz-user-focus: normal;
 }
 
-label[control] {
+label[control], label.radio-label, label.checkbox-label {
   -moz-binding: url("chrome://global/content/bindings/text.xml#label-control");
 }
 
+html|span.accesskey {
+  text-decoration: underline;
+}
+
 /********** textbox **********/
 
 textbox {