Bug 1399651 - Don't run the library animations in popup windows with toolbar=false. r?gijs draft
authorJared Wein <jwein@mozilla.com>
Wed, 13 Sep 2017 19:37:43 -0400
changeset 664465 f112038fc365e050c99669e1a0aeb97f5c98a464
parent 663831 1888ec2f277f6bb26271b8808e08914a21db9efe
child 731448 d24d92016e375ab819a57daff3971bc6f782f2d0
push id79700
push userbmo:jaws@mozilla.com
push dateWed, 13 Sep 2017 23:45:18 +0000
reviewersgijs
bugs1399651
milestone57.0a1
Bug 1399651 - Don't run the library animations in popup windows with toolbar=false. r?gijs MozReview-Commit-ID: 2vswZ2BTFwp
browser/base/content/browser-places.js
--- a/browser/base/content/browser-places.js
+++ b/browser/base/content/browser-places.js
@@ -1515,16 +1515,17 @@ var LibraryUI = {
         "toolkit.cosmeticAnimations.enabled", true);
     }
 
     let libraryButton = document.getElementById("library-button");
     if (!libraryButton ||
         libraryButton.getAttribute("cui-areatype") == "menu-panel" ||
         libraryButton.getAttribute("overflowedItem") == "true" ||
         !libraryButton.closest("#nav-bar") ||
+        !window.toolbar.visible ||
         !this.COSMETIC_ANIMATIONS_ENABLED) {
       return;
     }
 
     let animatableBox = document.getElementById("library-animatable-box");
     let navBar = document.getElementById("nav-bar");
     let libraryIcon = document.getAnonymousElementByAttribute(libraryButton, "class", "toolbarbutton-icon");
     let dwu = window.getInterface(Ci.nsIDOMWindowUtils);