Bug 1108998 - Add label to share provider buttons. r=Gijs, a=sledru
authorShane Caraveo <scaraveo@mozilla.com>
Wed, 10 Dec 2014 14:38:40 -0800
changeset 234283 27ff5a764eca5321bbaa81267b0033e9a9d31664
parent 234282 61340b38dced37931ad0b1bad933a6cd9002c9bc
child 234284 1bc888409400bac1afccce481a4d43187f3ad9f9
push id7354
push userryanvm@gmail.com
push dateFri, 12 Dec 2014 13:57:02 +0000
treeherdermozilla-aurora@993c07956f3d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersGijs, sledru
bugs1108998
milestone36.0a2
Bug 1108998 - Add label to share provider buttons. r=Gijs, a=sledru
browser/base/content/browser-social.js
--- a/browser/base/content/browser-social.js
+++ b/browser/base/content/browser-social.js
@@ -547,16 +547,17 @@ SocialShare = {
     for (let provider of providers) {
       let button = document.createElement("toolbarbutton");
       button.setAttribute("class", "toolbarbutton share-provider-button");
       button.setAttribute("type", "radio");
       button.setAttribute("group", "share-providers");
       button.setAttribute("image", provider.iconURL);
       button.setAttribute("tooltip", "share-button-tooltip");
       button.setAttribute("origin", provider.origin);
+      button.setAttribute("label", provider.name);
       button.setAttribute("oncommand", "SocialShare.sharePage(this.getAttribute('origin'));");
       if (provider == selectedProvider) {
         this.defaultButton = button;
       }
       hbox.insertBefore(button, addButton);
     }
     if (!this.defaultButton) {
       this.defaultButton = addButton;