Backed out changeset 81df156cb345 (
bug 1250203) for test failures in mochitest-5 test_ext_bookmarks.html, browser-chrome browser_views_liveupdate.js, OS X xpcshell test_browserGlue_migrate.js and maybe more. r=backout on a CLOSED TREE
Backed out changeset 81df156cb345 (
bug 1250203) for test failures in mochitest-5 test_ext_bookmarks.html, browser-chrome browser_views_liveupdate.js, OS X xpcshell test_browserGlue_migrate.js and maybe more. r=backout on a CLOSED TREE
--- a/browser/base/content/browser-places.js
+++ b/browser/base/content/browser-places.js
@@ -1402,43 +1402,16 @@ var BookmarkingUI = {
item.addEventListener("command", onItemCommand);
if (icon) {
item.setAttribute("image", icon);
}
fragment.appendChild(item);
}
root.containerOpen = false;
aHeaderItem.parentNode.insertBefore(fragment, aHeaderItem.nextSibling);
-
- // Update the menu when a bookmark is being removed.
- // The native menubar on Mac doesn't support live update, so this won't
- // work there.
- let observer = {
- onItemAdded() {},
- onBeginUpdateBatch() {},
- onEndUpdateBatch() {},
- onItemRemoved() {
- removeObserver();
- BookmarkingUI._updateRecentBookmarks(aHeaderItem, extraCSSClass);
- },
- onItemChanged() {},
- onItemVisited() {},
- onItemMoved() {}
- };
- PlacesUtils.bookmarks.addObserver(observer, false);
-
- let popup = aHeaderItem.parentNode;
- let removeObserver = function (event) {
- if (event && event.target != popup) {
- return;
- }
- PlacesUtils.bookmarks.removeObserver(observer);
- popup.removeEventListener("popuphidden", removeObserver);
- };
- popup.addEventListener("popuphidden", removeObserver);
},
/**
* Handles star styling based on page proxy state changes.
*/
onPageProxyStateChanged: function BUI_onPageProxyStateChanged(aState) {
if (!this._shouldUpdateStarState() || !this.star) {
return;