perf(onStart): Remove the call.

onStart should be called before any update so it should not be wrapped inside a .

README.md

@@ -248,7 +248,7 @@ $scope.slider = {
 
 **onlyBindHandles** - _Boolean (defaults to false)_: Set to true to only bind events on slider handles.
 
-**onStart** - _Function(sliderId)_: Function to be called when a slider update is started. If an id was set in the options, then it's passed to this callback.
+**onStart** - _Function(sliderId)_: Function to be called when a slider update is started. If an id was set in the options, then it's passed to this callback. This callback is called before any update on the model.
 
 **onChange** - _Function(sliderId)_: Function to be called when rz-slider-model or rz-slider-high change. If an id was set in the options, then it's passed to this callback.
 

dist/rzslider.js

@@ -795,10 +795,7 @@
        */
       callOnStart: function() {
         if (this.options.onStart) {
-          var self = this;
-          $timeout(function() {
-            self.options.onStart(self.options.id);
-          });
+          this.options.onStart(this.options.id);
         }
       },
 

src/rzslider.js

@@ -799,10 +799,7 @@
        */
       callOnStart: function() {
         if (this.options.onStart) {
-          var self = this;
-          $timeout(function() {
-            self.options.onStart(self.options.id);
-          });
+          this.options.onStart(this.options.id);
         }
       },
 

tests/spec/rz-slider-service-test.js

@@ -504,7 +504,6 @@ describe('rzslider - ', function() {
         createSlider(sliderConf);
 
         slider.callOnStart();
-        $timeout.flush();
         sliderConf.options.onStart.calledWith('test').should.be.true;
       });