Bug 682531 - Bookmarks Toolbar endless recursion (regression from Bug 677539). r+a=Neil
authorJens Hatlak <jh@junetz.de>
Mon, 29 Aug 2011 23:58:58 +0200
changeset 8795 ad5bc4ef1e4a7b8a41fa6a8883fe2fc270ae02f9
parent 8794 253dc0829ebdaf83760ef1cdab73b67ad081f69e
child 8796 092d1cb3a09175169fb9dd9e557cc8525fc83b6e
push id158
push userbugzilla@standard8.plus.com
push dateTue, 27 Sep 2011 19:18:14 +0000
treeherdercomm-beta@e47b99c61e4d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs682531, 677539
Bug 682531 - Bookmarks Toolbar endless recursion (regression from Bug 677539). r+a=Neil
suite/common/bookmarks/browser-places.js
--- a/suite/common/bookmarks/browser-places.js
+++ b/suite/common/bookmarks/browser-places.js
@@ -541,17 +541,17 @@ var BookmarksEventHandler = {
    */
   onCommand: function BEH_onCommand(aEvent) {
     var target = aEvent.originalTarget;
     if (target._placesNode)
       PlacesUIUtils.openNodeWithEvent(target._placesNode, aEvent);
   },
 
   onPopupShowing: function BEH_onPopupShowing(aEvent) {
-    if (!document.getElementById('bookmarksMenu')._placesView)
+    if (!aEvent.currentTarget.parentNode._placesView)
       new PlacesMenu(aEvent, 'place:folder=BOOKMARKS_MENU');
 
     document.getElementById("Browser:BookmarkAllTabs")
             .setAttribute("disabled", getBrowser().tabs.length == 1);
   },
   
   fillInBHTooltip: function BEH_fillInBHTooltip(aDocument, aEvent) {
     var node;