perf(animate): avoid unnecessary computations if animations are globally disabled

Author: gkalpak

src/ngAnimate/animateQueue.js

@@ -306,6 +306,16 @@ var $$AnimateQueueProvider = ['$animateProvider', /** @this */ function($animate
     return $animate;
 
     function queueAnimation(originalElement, event, initialOptions) {
+      // we create a fake runner with a working promise.
+      // These methods will become available after the digest has passed
+      var runner = new $$AnimateRunner();
+
+      // If animations are hard-disabled for the whole application there is no need to continue
+      if (!animationsEnabled) {
+        close();
+        return runner;
+      }
+
       // we always make a copy of the options since
       // there should never be any side effects on
       // the input data when running `$animateCss`.
@@ -317,10 +327,6 @@ var $$AnimateQueueProvider = ['$animateProvider', /** @this */ function($animate
 
       options = prepareAnimationOptions(options);
 
-      // we create a fake runner with a working promise.
-      // These methods will become available after the digest has passed
-      var runner = new $$AnimateRunner();
-
       // this is used to trigger callbacks in postDigest mode
       var runInNextPostDigestOrNow = postDigestTaskFactory();
 
@@ -348,7 +354,7 @@ var $$AnimateQueueProvider = ['$animateProvider', /** @this */ function($animate
         options.to = null;
       }
 
-      // there are situations where a directive issues an animation for
+      // There are situations where a directive issues an animation for
       // a jqLite wrapper that contains only comment nodes... If this
       // happens then there is no way we can perform an animation
       if (!node ||
@@ -362,12 +368,11 @@ var $$AnimateQueueProvider = ['$animateProvider', /** @this */ function($animate
 
       var documentHidden = $$isDocumentHidden();
 
-      // this is a hard disable of all animations for the application or on
-      // the element itself, therefore  there is no need to continue further
-      // past this point if not enabled
+      // This is a hard disable of all animations the element itself, therefore  there is no need to
+      // continue further past this point if not enabled
       // Animations are also disabled if the document is currently hidden (page is not visible
       // to the user), because browsers slow down or do not flush calls to requestAnimationFrame
-      var skipAnimations = !animationsEnabled || documentHidden || disabledElementsLookup.get(node);
+      var skipAnimations = documentHidden || disabledElementsLookup.get(node);
       var existingAnimation = (!skipAnimations && activeAnimationsLookup.get(node)) || {};
       var hasExistingAnimation = !!existingAnimation.state;