Bug 421464. The 'type' attribute on <style> should be optional. r=longsonr@gmail.com, sr=roc@ocallahan.org, blocking1.9=me

content/svg/content/src/nsSVGStyleElement.cpp

@@ -341,6 +341,9 @@ nsSVGStyleElement::GetStyleSheetInfo(nsAString& aTitle,
   ToLowerCase(aMedia);
 
   GetAttr(kNameSpaceID_None, nsGkAtoms::type, aType);
+  if (aType.IsEmpty()) {
+    aType.AssignLiteral("text/css");
+  }
 
   return;
 }

layout/reftests/svg/reftest.list

@@ -49,6 +49,7 @@ fails-if(MOZ_WIDGET_TOOLKIT=="cocoa") == opacity-and-gradient-01.svg pass.svg #
 == rootElement-null-01.svg pass.svg
 == stroke-width-percentage-01.svg pass.svg
 fails == style-property-not-on-script-element-01.svg pass.svg
+== style-without-type-attribute.svg pass.svg
 == svg-in-foreignObject-01.xhtml svg-in-foreignObject-01-ref.xhtml
 == svg-in-foreignObject-02.xhtml svg-in-foreignObject-01-ref.xhtml # reuse -01-ref.xhtml
 random-if(MOZ_WIDGET_TOOLKIT=="gtk2") == text-font-weight-01.svg text-font-weight-01-ref.svg # bug 386713

layout/reftests/svg/style-without-type-attribute.svg

@@ -0,0 +1,22 @@
+<!--
+     Any copyright is dedicated to the Public Domain.
+     http://creativecommons.org/licenses/publicdomain/
+-->
+<svg xmlns="http://www.w3.org/2000/svg" >
+
+  <title>Testcase for defaulting of 'type' attribute on &lt;style&gt;</title>
+
+  <!-- From https://bugzilla.mozilla.org/show_bug.cgi?id=421464 -->
+
+  <style>
+    #r1 { fill: lime; }
+  </style>
+
+  <style type="">
+    #r2 { fill: lime; }
+  </style>
+
+  <rect id="r1"         width="50%" height="100%" fill="red"/>
+  <rect id="r2" x="50%" width="50%" height="100%" fill="red"/>
+
+</svg>