Remove SVG constructor suppressions around foreignobject and altGlyph

tests/html/svgelement_test.dart

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library SvgElementTest;
 import 'dart:html';
 import 'dart:svg' as svg;
 import 'package:expect/expect.dart';
 import 'package:unittest/html_individual_config.dart';
 import 'package:unittest/unittest.dart';
@@ skipping 11 matching lines @@
         Element e = n;
         out.add(e.tagName);
       } else {
         out.add(n.text);
       }
     }
     return out;
   };
 
   testConstructor(String tagName, Function isExpectedClass,
-      [bool expectation = true]) {
+      [bool expectation = true, allowsInnerHtml = true]) {
     test(tagName, () {
       expect(isExpectedClass(new svg.SvgElement.tag(tagName)), expectation);
-      expect(isExpectedClass(
-          new svg.SvgElement.svg('<$tagName></$tagName>')), expectation);
+      if (allowsInnerHtml) {
+        expect(isExpectedClass(new svg.SvgElement.svg('<$tagName></$tagName>')),
+            allowsInnerHtml);
+      }
     });
   }
   group('additionalConstructors', () {
     test('valid', () {
       final svgContent = """
 <svg version="1.1">
   <circle></circle>
   <path></path>
 </svg>""";
       final el = new svg.SvgElement.svg(svgContent);
@@ skipping 189 matching lines @@
     });
   });
 
   group('supported_set', () {
     test('supported', () {
       expect(svg.SetElement.supported, true);
     });
   });
 
   group('constructors', () {
-    group('supported', () {
       testConstructor('a', (e) => e is svg.AElement);
       testConstructor('circle', (e) => e is svg.CircleElement);
       testConstructor('clipPath', (e) => e is svg.ClipPathElement);
       testConstructor('defs', (e) => e is svg.DefsElement);
       testConstructor('desc', (e) => e is svg.DescElement);
       testConstructor('ellipse', (e) => e is svg.EllipseElement);
       testConstructor('g', (e) => e is svg.GElement);
       testConstructor('image', (e) => e is svg.ImageElement);
       testConstructor('line', (e) => e is svg.LineElement);
       testConstructor('linearGradient', (e) => e is svg.LinearGradientElement);
@@ skipping 11 matching lines @@
       testConstructor('stop', (e) => e is svg.StopElement);
       testConstructor('style', (e) => e is svg.StyleElement);
       testConstructor('switch', (e) => e is svg.SwitchElement);
       testConstructor('symbol', (e) => e is svg.SymbolElement);
       testConstructor('tspan', (e) => e is svg.TSpanElement);
       testConstructor('text', (e) => e is svg.TextElement);
       testConstructor('textPath', (e) => e is svg.TextPathElement);
       testConstructor('title', (e) => e is svg.TitleElement);
       testConstructor('use', (e) => e is svg.UseElement);
       testConstructor('view', (e) => e is svg.ViewElement);
+      // TODO(alanknight): Issue 23144
       testConstructor('altGlyph', (e) => e is svg.AltGlyphElement,
-          svg.AltGlyphElement.supported);
+          svg.AltGlyphElement.supported, false);
       testConstructor('animate', (e) => e is svg.AnimateElement,
           svg.AnimateElement.supported);
       testConstructor('animateMotion', (e) => e is svg.AnimateMotionElement,
           svg.AnimateMotionElement.supported);
       testConstructor('animateTransform', (e) => e is svg.AnimateTransformElement,
           svg.AnimateTransformElement.supported);
       testConstructor('feBlend', (e) => e is svg.FEBlendElement,
           svg.FEBlendElement.supported);
       testConstructor('feColorMatrix', (e) => e is svg.FEColorMatrixElement,
           svg.FEColorMatrixElement.supported);
@@ skipping 38 matching lines @@
           svg.FESpecularLightingElement.supported);
       testConstructor('feSpotLight', (e) => e is svg.FESpotLightElement,
           svg.FESpotLightElement.supported);
       testConstructor('feTile', (e) => e is svg.FETileElement,
           svg.FETileElement.supported);
       testConstructor('feTurbulence', (e) => e is svg.FETurbulenceElement,
           svg.FETurbulenceElement.supported);
       testConstructor('filter', (e) => e is svg.FilterElement,
           svg.FilterElement.supported);
       testConstructor('foreignObject', (e) => e is svg.ForeignObjectElement,
-          svg.ForeignObjectElement.supported);
+          svg.ForeignObjectElement.supported, false);
       testConstructor('metadata', (e) => e is svg.MetadataElement);
       testConstructor('set', (e) => e is svg.SetElement,
           svg.SetElement.supported);
-    });
   });
 
   group('outerHtml', () {
     test('outerHtml', () {
       final el = new svg.SvgSvgElement();
       el.children.add(new svg.CircleElement());
       el.children.add(new svg.PathElement());
       expect([
         '<svg version="1.1"><circle></circle><path></path></svg>',
         '<svg xmlns="http://www.w3.org/2000/svg" version="1.1">'
@@ skipping 92 matching lines @@
   group('PathElement', () {
     test('pathSegList', () {
       svg.PathElement path =
           new svg.SvgElement.svg('<path d="M 100 100 L 300 100 L 200 300 z"/>');
       for (var seg in path.pathSegList) {
         expect(seg is svg.PathSeg, isTrue);
       }
     });
   });
 }