421464 - Make "type" attribute on <style> element optional

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Robert O'Callahan (:roc) (email my personal email if necessary)]

The SVG WG has decided the "type" attribute on the <style> element should be optional, to match HTML's <style> element. We should fix that, it should be easy.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

It is easy, and yes, we should do this.

Comment 2

[Robert Longson [:longsonr]]

Comment on attachment 307901 [details] [diff] [review]
patch

>+  if (aType.IsEmpty()) {
>+    aType += NS_LITERAL_STRING("text/css").get();

Isn't assignLiteral better/simpler?

aType.AssignLiteral("text/css")

r=longsonr with that, assuming that works.

>+  }
> 
>   return;
> }

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 307901 [details] [diff] [review]
patch

what Robert said

Comment 4

[Jonathan Watt [:jwatt]]

Yup.

Comment 5

[Jonathan Watt [:jwatt]]

Checked in.