Remove unimplemented SVG features

RESOLVED FIXED in mozilla21

Status

()

Core
SVG
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: dzbarsky, Assigned: dzbarsky)

Tracking

unspecified
mozilla21
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
It doesn't do anything useful.
(Assignee)

Comment 1

5 years ago
Having the implementations isn't useful and breaks feature testing.
Assignee: nobody → dzbarsky
Summary: Consider removing SVGElement.getPresentationAttribute → Remove unimplemented SVG features
(Assignee)

Comment 2

5 years ago
Created attachment 698529 [details] [diff] [review]
Patch
Attachment #698529 - Flags: review?(longsonr)

Comment 3

5 years ago
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+
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/9297bf41ab4d

Comment 5

5 years ago
https://hg.mozilla.org/mozilla-central/rev/9297bf41ab4d
Status: NEW → RESOLVED
Last Resolved: 5 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.