Bug 1399651 - Don't run the library animations in popup windows with toolbar=false. r=Gijs
authorJared Wein <jwein@mozilla.com>
Wed, 13 Sep 2017 19:37:43 -0400
changeset 430361 8d2ff3d80899ab9450a9b02b8cf974dabfb4d4a8
parent 430360 d5dc76a1482891edaced2f77d2ee86d58b55b29c
child 430362 14a79a4e94a2c78cff33b57376bf5d1441d9a7db
push id7761
push userjlund@mozilla.com
push dateFri, 15 Sep 2017 00:19:52 +0000
treeherdermozilla-beta@c38455951db4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersGijs
bugs1399651
milestone57.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 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);