Closed Bug 824218 Opened 11 years ago Closed 11 years ago

Remove unimplemented SVG features

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla21

People

(Reporter: dzbarsky, Assigned: dzbarsky)

Details

Attachments

(1 file)

It doesn't do anything useful.
Having the implementations isn't useful and breaks feature testing.
Assignee: nobody → dzbarsky
Summary: Consider removing SVGElement.getPresentationAttribute → Remove unimplemented SVG features
Attached patch PatchSplinter Review
Attachment #698529 - Flags: review?(longsonr)
Comment on attachment 698529 [details] [diff] [review]
Patch

>-/* nsIDOMCSSValue getPresentationAttribute (in DOMString name); */
>-NS_IMETHODIMP
>-nsSVGElement::GetPresentationAttribute(const nsAString& aName,
>-                                       nsIDOMCSSValue** aReturn)
>-{
>-  // Let's not implement this just yet. The CSSValue interface has been
>-  // deprecated by the CSS WG.
>-  // http://lists.w3.org/Archives/Public/www-style/2003Oct/0347.html

Please put the part of the comment starting from The CSSValue... 
in the webidl file so we don't lose it.

r=me with that.
Attachment #698529 - Flags: review?(longsonr) → review+
https://hg.mozilla.org/mozilla-central/rev/9297bf41ab4d
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
I've added this bug to the compatibility doc. Please correct the info if I'm wrong.
https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_21
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: