Bug 1229224: Another bustage fix for browser-social.js
authorDave Townsend <dtownsend@oxymoronical.com>
Mon, 04 Jan 2016 15:49:40 -0800
changeset 278328 11d71b79bc12b34d19c9b8aa469e64dd04e15ad9
parent 278327 1d259b3dbc70b520ffd62ae6dae536e75e7586d4
child 278329 c38424a542eb0e9149d4adabfe1db57bd390f398
push id29849
push usercbook@mozilla.com
push dateTue, 05 Jan 2016 10:57:04 +0000
treeherdermozilla-central@6e01570011a6 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1229224
milestone46.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1229224: Another bustage fix for browser-social.js
browser/base/content/browser-social.js
--- a/browser/base/content/browser-social.js
+++ b/browser/base/content/browser-social.js
@@ -1344,19 +1344,18 @@ SocialMarks = {
     }
   },
 
   getProviders: function() {
     // only rely on providers that the user has placed in the UI somewhere. This
     // also means that populateToolbarPalette must be called prior to using this
     // method, otherwise you get a big fat zero. For our use case with context
     // menu's, this is ok.
-    let tbh = this._toolbarHelper;
     return Social.providers.filter(p => p.markURL &&
-                                        document.getElementById(tbh.idFromOrigin(p.origin)));
+                                        document.getElementById(this._toolbarHelper.idFromOrigin(p.origin)));
   },
 
   populateContextMenu: function() {
     // only show a selection if enabled and there is more than one
     let providers = this.getProviders();
 
     // remove all previous entries by class
     let menus = [...document.getElementsByClassName("context-socialmarks")];