Bug 624961 - GroupItems.unregister() must remove groups from _arrangesPending[] [r=ian, a=beltzner]
authorTim Taubert <tim.taubert@gmx.de>
Tue, 18 Jan 2011 13:40:13 +0100
changeset 60824 018f911e4e4ca5a52390a652839354e27426d8d4
parent 60823 e95126669d5ed6170b478191d4bcea91fb677d38
child 60825 b659f70c8b42599b4b879c09fd9d0d747ae4d636
push id18123
push usereakhgari@mozilla.com
push dateTue, 18 Jan 2011 20:16:07 +0000
treeherdermozilla-central@5337fe8a5bfb [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersian, beltzner
bugs624961
milestone2.0b10pre
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 624961 - GroupItems.unregister() must remove groups from _arrangesPending[] [r=ian, a=beltzner]
browser/base/content/tabview/groupitems.js
--- a/browser/base/content/tabview/groupitems.js
+++ b/browser/base/content/tabview/groupitems.js
@@ -1945,16 +1945,20 @@ let GroupItems = {
   unregister: function GroupItems_unregister(groupItem) {
     var index = this.groupItems.indexOf(groupItem);
     if (index != -1)
       this.groupItems.splice(index, 1);
 
     if (groupItem == this._activeGroupItem)
       this._activeGroupItem = null;
 
+    this._arrangesPending = this._arrangesPending.filter(function (pending) {
+      return groupItem != pending.groupItem;
+    });
+
     UI.updateTabButton();
   },
 
   // ----------
   // Function: groupItem
   // Given some sort of identifier, returns the appropriate groupItem.
   // Currently only supports groupItem ids.
   groupItem: function GroupItems_groupItem(a) {