Bug 628051 - Remove killNewTabGroup; r=ian a=beltzner
authorMichael Yoshitaka Erlewine <mitcho@mitcho.com>
Tue, 25 Jan 2011 15:19:20 -0500
changeset 61282 c8ea68010263bf8963d532375625b656a55d0046
parent 61281 6253431ae03f5688ecdd8095fe5c235d640fab8b
child 61283 196efe55875452002d41e6e61a5e35a3eda99c0f
push id18292
push usereakhgari@mozilla.com
push dateTue, 25 Jan 2011 20:23:34 +0000
treeherdermozilla-central@0fb025a84958 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersian, beltzner
bugs628051
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 628051 - Remove killNewTabGroup; r=ian a=beltzner
browser/base/content/tabview/groupitems.js
--- a/browser/base/content/tabview/groupitems.js
+++ b/browser/base/content/tabview/groupitems.js
@@ -1933,17 +1933,16 @@ let GroupItems = {
   // ----------
   // Function: load
   // Loads the storage data for groups. 
   // Returns true if there was global group data.
   load: function GroupItems_load() {
     let groupItemsData = Storage.readGroupItemsData(gWindow);
     let groupItemData = Storage.readGroupItemData(gWindow);
     this.reconstitute(groupItemsData, groupItemData);
-    this.killNewTabGroup(); // temporary?
     
     return (groupItemsData && !Utils.isEmptyObject(groupItemsData));
   },
 
   // ----------
   // Function: groupItemStorageSanity
   // Given persistent storage data for a groupItem, returns true if it appears to not be damaged.
   groupItemStorageSanity: function GroupItems_groupItemStorageSanity(groupItemData) {
@@ -2341,31 +2340,16 @@ let GroupItems = {
       this._updateTabBar();
     else if (shouldShowTabView) {
       tab._tabViewTabItem.setZoomPrep(false);
       UI.showTabView();
     }
   },
 
   // ----------
-  // Function: killNewTabGroup
-  // Removes the New Tab Group, which is now defunct. See bug 575851 and comments therein.
-  killNewTabGroup: function GroupItems_killNewTabGroup() {
-    // not localized as the original "New Tabs" group title was never localized
-    // to begin with
-    let newTabGroupTitle = "New Tabs";
-    this.groupItems.forEach(function(groupItem) {
-      if (groupItem.getTitle() == newTabGroupTitle && groupItem.locked.title) {
-        groupItem.removeAll();
-        groupItem.close();
-      }
-    });
-  },
-
-  // ----------
   // Function: removeHiddenGroups
   // Removes all hidden groups' data and its browser tabs.
   removeHiddenGroups: function GroupItems_removeHiddenGroups() {
     if (this._removingHiddenGroups)
       return;
     this._removingHiddenGroups = true;
 
     let groupItems = this.groupItems.concat();