fix(NgAttrMustacheDirective): support parsing of multiline attribute values

Author: mraleph

lib/core_dom/ng_mustache.dart

@@ -21,13 +21,12 @@ class NgTextMustacheDirective {
 
 @NgDirective(selector: r'[*=/{{.*}}/]')
 class NgAttrMustacheDirective {
-  static RegExp ATTR_NAME_VALUE_REGEXP = new RegExp(r'^([^=]+)=(.*)$');
-
-// This Directive is special and does not go through injection.
+  // This Directive is special and does not go through injection.
   NgAttrMustacheDirective(NodeAttrs attrs, String markup, Interpolate interpolate, Scope scope) {
-    var match = ATTR_NAME_VALUE_REGEXP.firstMatch(markup);
-    var attrName = match[1];
-    Interpolation interpolation = interpolate(match[2]);
+    var eqPos = markup.indexOf('=');
+    var attrName = markup.substring(0, eqPos);
+    var attrValue = markup.substring(eqPos + 1);
+    Interpolation interpolation = interpolate(attrValue);
     interpolation.setter = (text) => attrs[attrName] = text;
     interpolation.setter('');
     scope.$watchSet(interpolation.watchExpressions, interpolation.call, markup.trim());

test/core_dom/ng_mustache_spec.dart

@@ -50,6 +50,22 @@ main() {
       expect(element.attr('some-attr')).toEqual('OK');
       expect(element.attr('other-attr')).toEqual('23');
     }));
+
+
+    it('should allow newlines in attribute', inject((Compiler $compile, Scope $rootScope, Injector injector) {
+      var element = $('<div multiline-attr="line1: {{line1}}\nline2: {{line2}}"></div>');
+      var template = $compile(element);
+
+      $rootScope.line1 = 'L1';
+      $rootScope.line2 = 'L2';
+      var block = template(injector);
+
+      element = $(block.elements);
+
+      expect(element.attr('multiline-attr')).toEqual('');
+      $rootScope.$digest();
+      expect(element.attr('multiline-attr')).toEqual('line1: L1\nline2: L2');
+    }));
   });
 
   describe('NgShow', () {