Bug 579869 - Skip the tab opening animation if the new tab has been pinned. r=gavin

--HG--
extra : rebase_source : 4406ddd65a67e9c65705f65172afe6af1ded1c17

browser/base/content/browser.css

@@ -31,7 +31,7 @@ tabbrowser {
   width: 0;
 }
 
-.tabbrowser-tab:not([fadein]) {
+.tabbrowser-tab:not([pinned]):not([fadein]) {
   max-width: 1px;
   min-width: 1px;
 }
@@ -40,9 +40,9 @@ tabbrowser {
   -moz-transition: min-width .2s ease-out, max-width .25s ease-out;
 }
 
-.tabbrowser-tab:not([fadein]) > .tab-text,
-.tabbrowser-tab:not([fadein]) > .tab-icon-image,
-.tabbrowser-tab:not([fadein]) > .tab-close-button {
+.tabbrowser-tab:not([fadein]):not([pinned]) > .tab-text,
+.tabbrowser-tab:not([fadein]):not([pinned]) > .tab-icon-image,
+.tabbrowser-tab:not([fadein]):not([pinned]) > .tab-close-button {
   opacity: 0 !important;
 }
 

browser/base/content/tabbrowser.xml

@@ -187,6 +187,7 @@
             return;
 
           this.moveTabTo(aTab, this._numPinnedTabs - 1);
+          aTab.setAttribute("fadein", "true");
           aTab.removeAttribute("pinned");
           aTab.style.MozMarginStart = "";
           this.tabContainer._positionPinnedTabs();
@@ -1102,7 +1103,12 @@
                 tabContainer._handleNewTab(t);
               }, 0, this.tabContainer);
             } else {
-              setTimeout(function () { t.setAttribute("fadein", "true"); }, 0);
+              setTimeout(function (tabContainer) {
+                if (t.pinned)
+                  tabContainer._handleNewTab(t);
+                else
+                  t.setAttribute("fadein", "true");
+              }, 0, this.tabContainer);
             }
 
             this.tabContainer.appendChild(t);