Pass invokeApply == false into $interval

This prevents excessive $digest cycles whenever the $interval callback
fires. The invokeApply param is effectively ignored in 1.2.x so this
patch doesn't do anything there.

See angular/angular.js#7103 for more details
on the fix that landed in 1.3.x.

Author: Marvin Tam

src/angular-idle.js

@@ -141,7 +141,7 @@
           stopKeepalive();
           state.countdown = options.warningDuration;
           countdown();
-          state.warning = $interval(countdown, 1000, options.warningDuration);
+          state.warning = $interval(countdown, 1000, options.warningDuration, false);
         } else {
           startKeepalive();
         }
@@ -178,7 +178,7 @@
 
           state.running = true;
 
-          state.idle = $interval(toggleState, options.idleDuration * 1000);
+          state.idle = $interval(toggleState, options.idleDuration * 1000, 0, false);
         },
         unwatch: function() {
           $interval.cancel(state.idle);

src/angular-idle.spec.js

@@ -11,7 +11,7 @@ describe('ngIdle', function() {
   });
 
   describe('idle', function() {
-    var $idleProvider, $interval, $rootScope, $log, $document, $keepalive;
+    var $idleProvider, $interval, $rootScope, $log, $document, $keepalive, $timeout;
     var DEFAULTIDLEDURATION = 20*60*1000, DEFAULTWARNINGDURATION = 30 * 1000;
 
     beforeEach(module('ngIdle.idle'));
@@ -26,11 +26,12 @@ describe('ngIdle', function() {
 
       module('app');
 
-      inject(function(_$interval_, _$log_, _$rootScope_, _$document_) {
+      inject(function(_$interval_, _$log_, _$rootScope_, _$document_, _$timeout_) {
         $rootScope = _$rootScope_;
         $interval = _$interval_;
         $log = _$log_;
         $document = _$document_;
+        $timeout = _$timeout_;
       });
 
       $keepalive = {
@@ -164,6 +165,7 @@ describe('ngIdle', function() {
         $idle.watch();
 
         $interval.flush(DEFAULTIDLEDURATION);
+        $timeout.flush();
 
         expect($rootScope.$broadcast).toHaveBeenCalledWith('$idleStart');
         expect($keepalive.stop).toHaveBeenCalled();
@@ -175,6 +177,7 @@ describe('ngIdle', function() {
         $idle.watch();
 
         $interval.flush(DEFAULTIDLEDURATION);
+        $timeout.flush();
 
         $idle.watch();
 
@@ -189,15 +192,24 @@ describe('ngIdle', function() {
         $idle.watch();
 
         $interval.flush(DEFAULTIDLEDURATION);
+        $timeout.flush();
 
         expect($rootScope.$broadcast).toHaveBeenCalledWith('$idleStart');
         expect($rootScope.$broadcast).toHaveBeenCalledWith('$idleWarn', 3);
+
         $interval.flush(1000);
+        $timeout.flush();
+
         expect($rootScope.$broadcast).toHaveBeenCalledWith('$idleWarn', 2);
+
         $interval.flush(1000);
+        $timeout.flush();
+
         expect($rootScope.$broadcast).toHaveBeenCalledWith('$idleWarn', 1);
 
         $interval.flush(1000);
+        $timeout.flush();
+
         expect($rootScope.$broadcast).toHaveBeenCalledWith('$idleTimeout');
 
         // ensure idle interval doesn't keep executing after $idleStart
@@ -214,6 +226,7 @@ describe('ngIdle', function() {
         $interval.flush(DEFAULTIDLEDURATION);
 
         $interval.flush(1000);
+        $timeout.flush();
 
         expect($idle.idling()).toBe(true);