fix(ngInclude): correctly add non-html namespaced template content

It is now possible for ngInclude to correctly load SVG content in non-blink browsers, which do not
sort out the namespace when parsing HTML.

Closes angular#7538

Author: caitp

src/ng/directive/ngInclude.js

@@ -267,8 +267,10 @@ var ngIncludeFillContentDirective = ['$compile',
       priority: -400,
       require: 'ngInclude',
       link: function(scope, $element, $attr, ctrl) {
-        $element.html(ctrl.template);
-        $compile($element.contents())(scope);
+        $element.empty();
+        $compile(ctrl.template)(scope, function namespaceAdaptedClone(clone) {
+          $element.append(clone);
+        }, undefined, undefined, $element);
       }
     };
   }];

test/ng/directive/ngIncludeSpec.js

@@ -590,6 +590,27 @@ describe('ngInclude and transcludes', function() {
       expect(root[0]).toBe(element[0]);
     });
   });
+
+
+  it('should correctly work with SVG content', function() {
+    if (!window.SVGRectElement) return;
+    module(function() {
+      directive('test', valueFn({
+        templateNamespace: 'svg',
+        templateUrl: 'my-rect.html',
+        replace: true
+      }));
+    });
+    inject(function($compile, $rootScope, $httpBackend) {
+      $httpBackend.expectGET('my-rect.html').respond('<g ng-include="\'include.svg\'"></g>');
+      $httpBackend.expectGET('include.svg').respond('<rect></rect>');
+      element = $compile('<svg><test></test></svg>')($rootScope);
+      $httpBackend.flush();
+      var child = element.find('rect');
+      expect(child.length).toBe(1);
+      expect(child[0] instanceof SVGRectElement).toBe(true);
+    });
+  });
 });
 
 describe('ngInclude animations', function() {