fix(ngValue): set value property instead of attr

in case of input element set value using property instead of attribute
fixed #13984

Author: m-amr

src/.jshintrc

@@ -140,6 +140,7 @@
     "jqLiteParseHTML": false,
     "jqLiteWrapNode": false,
     "getBooleanAttrName": false,
+    "getValueAttrName": false,
     "getAliasedAttrName": false,
     "createEventHandler": false,
     "JQLitePrototype": false,

src/jqLite.js

@@ -15,6 +15,7 @@
   addEventListenerFn: true,
   removeEventListenerFn: true,
   BOOLEAN_ATTR: true,
+  VALUE_ATTR: true,
   ALIASED_ATTR: true,
 */
 
@@ -577,6 +578,15 @@ var BOOLEAN_ELEMENTS = {};
 forEach('input,select,option,textarea,button,form,details'.split(','), function(value) {
   BOOLEAN_ELEMENTS[value] = true;
 });
+
+var VALUE_ATTR = {};
+VALUE_ATTR['value'] = 'value';
+
+var VALUE_ELEMENTS = {};
+forEach('input,textarea'.split(','), function(value) {
+  VALUE_ELEMENTS[value] = true;
+});
+
 var ALIASED_ATTR = {
   'ngMinlength': 'minlength',
   'ngMaxlength': 'maxlength',
@@ -593,6 +603,14 @@ function getBooleanAttrName(element, name) {
   return booleanAttr && BOOLEAN_ELEMENTS[nodeName_(element)] && booleanAttr;
 }
 
+function getValueAttrName(element, name) {
+  // check dom last since we will most likely fail on name
+  var valueAttr = VALUE_ATTR[name.toLowerCase()];
+
+  // booleanAttr is here twice to minimize DOM access
+  return valueAttr && VALUE_ELEMENTS[nodeName_(element)] && valueAttr;
+}
+
 function getAliasedAttrName(name) {
   return ALIASED_ATTR[name];
 }

src/ng/compile.js

@@ -1322,13 +1322,14 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
 
         var node = this.$$element[0],
             booleanKey = getBooleanAttrName(node, key),
+            valueKey = getValueAttrName(node, key),
             aliasedKey = getAliasedAttrName(key),
             observer = key,
             nodeName;
 
-        if (booleanKey) {
+        if (booleanKey || valueKey) {
           this.$$element.prop(key, value);
-          attrName = booleanKey;
+          attrName = booleanKey ? booleanKey : valueKey;
         } else if (aliasedKey) {
           this[aliasedKey] = value;
           observer = aliasedKey;

test/ng/directive/inputSpec.js

@@ -3046,6 +3046,26 @@ describe('input', function() {
       expect(inputElm[0].getAttribute('value')).toBe('something');
     });
 
+    it('should update the input "value" property and attribute after change the "value" property', function() {
+      var inputElm = helper.compileInput('<input type="text" ng-value="value">');
+
+      $rootScope.$apply(function() {
+        $rootScope.value = 'something';
+      });
+      expect(inputElm[0].value).toBe('something');
+      expect(inputElm[0].getAttribute('value')).toBe('something');
+
+      helper.changeInputValueTo('newValue');
+
+      $rootScope.$apply(function() {
+        $rootScope.value = 'anotherValue';
+      });
+      expect(inputElm[0].value).toBe('anotherValue');
+      expect(inputElm[0].getAttribute('value')).toBe('anotherValue');
+
+
+    });
+
 
     it('should evaluate and set constant expressions', function() {
       var inputElm = helper.compileInput('<input type="radio" ng-model="selected" ng-value="true">' +