From 4031df8b623ffdd8ffdada3947365a05f1eb6270 Mon Sep 17 00:00:00 2001
From: Ian Gilman <ian@iangilman.com>
Date: Mon, 21 Jun 2010 16:35:29 -0700
Subject: [PATCH] + Raymond's close button fix missed a couple cases: creating
 new tabs with command+click, and closing non-selected tabs with the close
 buttons in the tabbar. Fixed.

---
 browser/base/content/tabcandy/app/ui.js | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/browser/base/content/tabcandy/app/ui.js b/browser/base/content/tabcandy/app/ui.js
index d0f4c539c80..2c71bec5aab 100644
--- a/browser/base/content/tabcandy/app/ui.js
+++ b/browser/base/content/tabcandy/app/ui.js
@@ -241,10 +241,10 @@ window.Page = {
   },
   
   setCloseButtonOnTabs : function() {
-    // XXX: we will need to modify the adjustTabstrip() to fix this when merging
+    // TODO: we will need to modify the adjustTabstrip() to fix this when merging
     // this extension to Firefox.
     // http://mxr.mozilla.org/mozilla1.9.2/source/browser/base/content/tabbrowser.xml#3050
-    setTimeout(function() {
+    iQ.timeout(function() { // iQ.timeout adds a try/catch to setTimeout
       var tabContainer = Utils.getCurrentWindow().gBrowser.tabContainer;
       if (tabContainer.mCloseButtons == 1 &&
           tabContainer.getAttribute("overflow") != "true") {
@@ -320,10 +320,6 @@ window.Page = {
   // ----------  
   init: function() {
     var self = this;
-/*  Ian suspects we don't need these lines
-    Utils.homeTab.raw.maxWidth = 60;
-    Utils.homeTab.raw.minWidth = 60;
-*/
         
     // When you click on the background/empty part of TabCandy
     // we create a new group.
@@ -336,6 +332,8 @@ window.Page = {
         
     Tabs.onClose(function(){
       iQ.timeout(function() { // Marshal event from chrome thread to DOM thread
+        Page.setCloseButtonOnTabs();        
+          
         // Only go back to the TabCandy tab when there you close the last
         // tab of a group.
         var group = Groups.getActiveGroup();
@@ -609,6 +607,7 @@ UIClass.prototype = {
       Tabs.onOpen(function(a, b) {
         iQ.timeout(function() { // Marshal event from chrome thread to DOM thread
           self.navBar.show();
+          Page.setCloseButtonOnTabs();
         }, 1);
       });