fixup! fix($compile): fix ng-prop-* with undefined values

Author: jbedard

src/ng/compile.js

@@ -3841,7 +3841,11 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
                 if (propValue !== undefined) {
                   $element.prop(propName, propValue);
                 } else {
-                  delete $element[0][propName];
+                  if ($element.removeProp) {
+                    $element.removeProp(propName);
+                  } else {
+                    delete $element[0][propName];
+                  }
                 }
               }
 

test/ng/ngPropSpec.js

@@ -90,6 +90,23 @@ describe('ngProp*', function() {
     expect(element[0].hasOwnProperty('text')).toBe(false);
   }));
 
+  // Ensure jQuery prop/removeProp are used to support fix/hooks
+  if (angular.element.prototype.removeProp) {
+    it('should use jQuery.fn.prop() and jQuery.fn.removeProp() to support jQuery.propFix/Hooks', inject(function($rootScope, $compile) {
+      $compile('<span ng-prop-text="myText" />')($rootScope);
+      spyOn(angular.element.fn, 'prop').and.callThrough();
+      spyOn(angular.element.fn, 'removeProp').and.callThrough();
+
+      $rootScope.myText = 'a value';
+      $rootScope.$digest();
+      expect(angular.element.fn.prop).toHaveBeenCalledWith('text', 'a value');
+
+      $rootScope.myText = undefined;
+      $rootScope.$digest();
+      expect(angular.element.fn.removeProp).toHaveBeenCalledWith('text');
+    }));
+  }
+
   it('should support mixed case using underscore-separated names', inject(function($rootScope, $compile) {
     var element = $compile('<span ng-prop-a_bcd_e="value" />')($rootScope);
     $rootScope.value = 123;