Bug 527986: new tabs open with partial thumbnail of old tab [r=gavin.sharp]
authorVivien Nicolas <21@vingtetun>
Sun, 22 Nov 2009 23:27:31 -0500
changeset 1143 8827852cdf72a1ac87c1d0c448cfab98eab4e431
parent 1142 875f06cd2ffcf3b1ac883df5ee75b29ee1cfc9a0
child 1144 bc1395dbbbcbc15114f1d24a2270dcfdc678b7bb
push id1010
push usermfinkle@mozilla.com
push dateMon, 23 Nov 2009 04:27:56 +0000
reviewersgavin.sharp
bugs527986
Bug 527986: new tabs open with partial thumbnail of old tab [r=gavin.sharp]
chrome/content/browser.js
themes/hildon/browser.css
themes/wince/browser.css
--- a/chrome/content/browser.js
+++ b/chrome/content/browser.js
@@ -2329,22 +2329,21 @@ ProgressController.prototype = {
     this._tab.endLoading();
 
     if (this._tab == Browser.selectedTab) {
       BrowserUI.update(TOOLBARSTATE_LOADED);
       this.browser.docShell.isOffScreenBrowser = true;
     }
 
     // if we are idle at this point, be sure to kick start the prefetcher
-    if (Browser._browserView._idleServiceObserver.isIdle()) {
-	//dump("kicking off restartPrefetchCrawl\n");
-	Browser._browserView._tileManager.restartPrefetchCrawl();
-    }
-
-    this._tab.updateThumbnail();
+    if (Browser._browserView._idleServiceObserver.isIdle())
+      Browser._browserView._tileManager.restartPrefetchCrawl();
+    
+    if (this.browser.currentURI.spec != "about:blank")
+      this._tab.updateThumbnail();
   },
 
   _documentStop: function() {
     // translate any phone numbers
     Browser.translatePhoneNumbers();
 
     if (this._tab == Browser.selectedTab && !BrowserUI.isAutoCompleteOpen()) {
       // focus the dom window
@@ -2509,19 +2508,16 @@ Tab.prototype = {
   },
 
   /** Returns tab's identity state for updating security UI. */
   getIdentityState: function() {
     return this._listener.state;
   },
 
   startLoading: function() {
-    //if (this._loading)
-    //  dump("!!! Already loading this tab, please file a bug\n");
-
     this._loading = true;
     this._browserViewportState.zoomChanged = false;
 
     if (!this._loadingTimeout) {
       Browser._browserView.beginBatchOperation();
       Browser._browserView.invalidateEntireView();
       this._loadingTimeout = setTimeout(Util.bind(this._resizeAndPaint, this), 2000);
     }
--- a/themes/hildon/browser.css
+++ b/themes/hildon/browser.css
@@ -695,17 +695,18 @@ box[type="documenttab"] {
   /* display:block allow us to change the line-height, it won't work otherwise */
   display: block;
   width: 128px;
   line-height: 0;
 }
 
 box[type="documenttab"] .documenttab-canvas {
   /* keep the unselected thumbnails aligned with the selected one */
-  border: 0.8mm solid transparent;
+  border: 0.8mm solid #36373b;
+  background-color: white;
 }
 
 box[type="documenttab"][selected="true"] .documenttab-canvas {
   border: 0.8mm solid #262629;
   -moz-border-radius: 3px;
 }
 
 box[type="documenttab"] .documenttab-close {
--- a/themes/wince/browser.css
+++ b/themes/wince/browser.css
@@ -440,21 +440,22 @@ autocompleteresult.allbookmarks > .autoc
 
 #tabs-controls {
   margin-top: 1.1mm; /* core spacing */
   -moz-box-pack: start;
 }
 
 box[type="documenttab"] .documenttab-canvas {
   /* keep the unselected thumbnails aligned with the selected one */
-  border: 1.0mm solid transparent;
+  border: 0.8mm solid #36373b;
+  background-color: white;
 }
 
 box[type="documenttab"][selected="true"] .documenttab-canvas {
-  border: 1.0mm solid #262629;
+  border: 0.8mm solid #262629;
   -moz-border-radius: 4px;
 }
 
 box[type="documenttab"]:only-child .documenttab-close {
   display: none;
 }
 
 /* bookmark editor   ------------------------------------------------------- */