Bug 1400337 - only set 'progress' attribute if 'busy' is already set, r?dao draft
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Sat, 16 Sep 2017 21:42:41 +0100
changeset 665959 449613405e72edaccc5c60f55e9ef234de3971ee
parent 665900 27e7b4261e4b5a016e09adb5332a181e43ab520e
child 731942 e43d1d743965ac74ca3acae050b5f405dd1e397c
push id80236
push usergijskruitbosch@gmail.com
push dateSat, 16 Sep 2017 20:43:11 +0000
reviewersdao
bugs1400337
milestone57.0a1
Bug 1400337 - only set 'progress' attribute if 'busy' is already set, r?dao MozReview-Commit-ID: Ivumb7ze69A
browser/base/content/tabbrowser.xml
--- a/browser/base/content/tabbrowser.xml
+++ b/browser/base/content/tabbrowser.xml
@@ -664,17 +664,17 @@
             onProgressChange(aWebProgress, aRequest,
                              aCurSelfProgress, aMaxSelfProgress,
                              aCurTotalProgress, aMaxTotalProgress) {
               this.mTotalProgress = aMaxTotalProgress ? aCurTotalProgress / aMaxTotalProgress : 0;
 
               if (!this._shouldShowProgress(aRequest))
                 return;
 
-              if (this.mTotalProgress && aWebProgress && aWebProgress.isTopLevel)
+              if (this.mTotalProgress && this.mTab.hasAttribute("busy"))
                 this.mTab.setAttribute("progress", "true");
 
               this._callProgressListeners("onProgressChange",
                                           [aWebProgress, aRequest,
                                            aCurSelfProgress, aMaxSelfProgress,
                                            aCurTotalProgress, aMaxTotalProgress]);
             },