From 7480f431c266f8daf04175190b65cfd9f123caf4 Mon Sep 17 00:00:00 2001
From: Caitlin Potter <caitpotter88@gmail.com>
Date: Tue, 4 Nov 2014 17:46:29 -0500
Subject: [PATCH] fix(input): don't invoke viewChangeListeners before real user
 input

Closes #9867
---
 src/ng/directive/input.js      |  9 +++++++--
 test/ng/directive/inputSpec.js | 12 ++++++++++++
 2 files changed, 19 insertions(+), 2 deletions(-)

diff --git a/src/ng/directive/input.js b/src/ng/directive/input.js
index 639afcb9dfcb..9bd9767d1dae 100644
--- a/src/ng/directive/input.js
+++ b/src/ng/directive/input.js
@@ -2122,13 +2122,18 @@ var NgModelController = ['$scope', '$exceptionHandler', '$attrs', '$element', '$
 
     function writeToModelIfNeeded() {
       if (ctrl.$modelValue !== prevModelValue) {
-        ctrl.$$writeModelToScope();
+        ctrl.$$writeModelToScope(prevModelValue);
       }
     }
   };
 
-  this.$$writeModelToScope = function() {
+  this.$$writeModelToScope = function(oldVal) {
     ngModelSet(ctrl.$modelValue);
+    // String input types can transform a value into a string without touching it.
+    if (oldVal != null && !isNaN(oldVal) &&
+        ctrl.$modelValue === oldVal.toString()) {
+      return;
+    }
     forEach(ctrl.$viewChangeListeners, function(listener) {
       try {
         listener();
diff --git a/test/ng/directive/inputSpec.js b/test/ng/directive/inputSpec.js
index 8d4e47763609..da8606ed54b3 100644
--- a/test/ng/directive/inputSpec.js
+++ b/test/ng/directive/inputSpec.js
@@ -1349,6 +1349,18 @@ describe('input', function() {
   });
 
 
+  it('should not invoke viewChangeListeners before input is touched', function() {
+    scope.value = 1;
+    var change = scope.change = jasmine.createSpy('change');
+    var element = $compile('<div><div ng-repeat="i in [1]">' +
+          '<input type="text" ng-model="value" maxlength="1" ng-change="change()" />' +
+        '</div></div>')(scope);
+    scope.$digest();
+    expect(change).not.toHaveBeenCalled();
+    dealoc(element);
+  });
+
+
   describe('compositionevents', function() {
     it('should not update the model between "compositionstart" and "compositionend" on non android', inject(function($sniffer) {
       $sniffer.android = false;