Bug 1036646 fix double command handling for socialmarks button, r=Gijs
authorShane Caraveo <scaraveo@mozilla.com>
Wed, 09 Jul 2014 16:40:26 -0700
changeset 193281 b576014a4876104e47f60109235e06e2631774af
parent 193280 d4f97168453a91922d42dd177db6ca631cd228b0
child 193282 64c6782c7ca95aac1011abe7f1e1e95b916ee915
push id1
push userroot
push dateMon, 20 Oct 2014 17:29:22 +0000
reviewersGijs
bugs1036646
milestone33.0a1
Bug 1036646 fix double command handling for socialmarks button, r=Gijs
browser/modules/Social.jsm
--- a/browser/modules/Social.jsm
+++ b/browser/modules/Social.jsm
@@ -304,17 +304,16 @@ function CreateSocialMarkWidget(aId, aPr
     defaultArea: CustomizableUI.AREA_NAVBAR,
     onBuild: function(aDocument) {
       let node = aDocument.createElement('toolbarbutton');
       node.id = this.id;
       node.setAttribute('class', 'toolbarbutton-1 chromeclass-toolbar-additional social-mark-button');
       node.setAttribute('type', "socialmark");
       node.style.listStyleImage = "url(" + (aProvider.unmarkedIcon || aProvider.icon32URL || aProvider.iconURL) + ")";
       node.setAttribute("origin", aProvider.origin);
-      node.setAttribute("oncommand", "this.markCurrentPage();");
 
       let window = aDocument.defaultView;
       let menuLabel = window.gNavigatorBundle.getFormattedString("social.markpageMenu.label", [aProvider.name]);
       node.setAttribute("label", menuLabel);
       node.setAttribute("tooltiptext", menuLabel);
 
       return node;
     }