Bug 1423220 - Don't delay activating the DocShell for tabs that we're rendering by switching and not warming. r=dthayer
authorMike Conley <mconley@mozilla.com>
Fri, 16 Mar 2018 11:39:22 -0400
changeset 408630 8ff03a020b697ba978a0298f9396b38b1f9f069e
parent 408629 d11b63f72c883a2961fb8898e3ac5ef58419b5cb
child 408631 beae66d4b661a78c53baa8a5c766975743577a79
push id100996
push userbtara@mozilla.com
push dateSat, 17 Mar 2018 10:37:43 +0000
treeherdermozilla-inbound@97160a734959 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdthayer
bugs1423220
milestone61.0a1
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 1423220 - Don't delay activating the DocShell for tabs that we're rendering by switching and not warming. r=dthayer MozReview-Commit-ID: 3UmocajaSWJ
browser/modules/AsyncTabSwitcher.jsm
--- a/browser/modules/AsyncTabSwitcher.jsm
+++ b/browser/modules/AsyncTabSwitcher.jsm
@@ -249,17 +249,19 @@ class AsyncTabSwitcher {
 
     this.setTabStateNoAction(tab, state);
 
     let browser = tab.linkedBrowser;
     let { tabParent } = browser.frameLoader;
     if (state == this.STATE_LOADING) {
       this.assert(!this.minimizedOrFullyOccluded);
 
-      if (!this.tabbrowser.tabWarmingEnabled) {
+      // If we're not in the process of warming this tab, we
+      // don't need to delay activating its DocShell.
+      if (!this.warmingTabs.has(tab)) {
         browser.docShellIsActive = true;
       }
 
       if (tabParent) {
         browser.renderLayers = true;
       } else {
         this.onLayersReady(browser);
       }