fixup! feat($compile): add support for arbitrary property and event bindings

Author: jbedard

src/ng/compile.js

@@ -3461,6 +3461,9 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
       return PROP_CONTEXTS[tag + "|" + prop] || PROP_CONTEXTS["*|" + prop];
     }
 
+    function sanitizeSrcsetPropertyValue(value) {
+      return sanitizeSrcset($sce.valueOf(value), 'ng-prop-srcset');
+    }
     function addPropertyDirective(node, directives, attrName, propName) {
       if (EVENT_HANDLER_ATTR_REGEXP.test(propName)) {
         throw $compileMinErr('nodomevents',
@@ -3471,23 +3474,22 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
       var nodeName = nodeName_(node);
       var trustedContext = getTrustedPropContext(nodeName, propName);
 
+      var sanitizer = identity;
+      // Sanitize img[srcset] + source[srcset] values.
+      if ((nodeName === 'img' || nodeName === 'source') && propName === 'srcset') {
+        sanitizer = sanitizeSrcsetPropertyValue;
+      } else if (trustedContext) {
+        sanitizer = $sce.getTrusted.bind($sce, trustedContext);
+      }
+
       directives.push({
         priority: 100,
         compile: function ngPropCompileFn(_, attr) {
           var fn = $parse(attr[attrName]);
           return {
             pre: function ngPropPreLinkFn(scope, $element) {
               scope.$watch(fn, function propertyWatchActionFn(value) {
-                if (value) {
-                  // Sanitize img[srcset] + source[srcset] values.
-                  if ((nodeName === 'img' || nodeName === 'source') && propName === 'srcset') {
-                    value = sanitizeSrcset($sce.valueOf(value), 'ng-prop-srcset');
-                  } else if (trustedContext) {
-                    value = $sce.getTrusted(trustedContext, value);
-                  }
-                }
-
-                $element.prop(propName, value);
+                $element.prop(propName, value && sanitizer(value));
               });
             }
           };