Last Comment Bug 824218 - Remove unimplemented SVG features
: Remove unimplemented SVG features
Product: Core
Classification: Components
Component: SVG (show other bugs)
: unspecified
: All All
-- normal (vote)
: mozilla21
Assigned To: David Zbarsky (:dzbarsky)
: Jet Villegas (:jet)
Depends on:
  Show dependency treegraph
Reported: 2012-12-21 21:58 PST by David Zbarsky (:dzbarsky)
Modified: 2013-02-23 23:32 PST (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch (29.84 KB, patch)
2013-01-06 21:12 PST, David Zbarsky (:dzbarsky)
longsonr: review+
Details | Diff | Splinter Review

Description User image David Zbarsky (:dzbarsky) 2012-12-21 21:58:41 PST
It doesn't do anything useful.
Comment 1 User image David Zbarsky (:dzbarsky) 2013-01-06 21:06:31 PST
Having the implementations isn't useful and breaks feature testing.
Comment 2 User image David Zbarsky (:dzbarsky) 2013-01-06 21:12:57 PST
Created attachment 698529 [details] [diff] [review]
Comment 3 User image Robert Longson 2013-01-07 07:33:54 PST
Comment on attachment 698529 [details] [diff] [review]

>-/* nsIDOMCSSValue getPresentationAttribute (in DOMString name); */
>-nsSVGElement::GetPresentationAttribute(const nsAString& aName,
>-                                       nsIDOMCSSValue** aReturn)
>-  // Let's not implement this just yet. The CSSValue interface has been
>-  // deprecated by the CSS WG.
>-  //

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.
Comment 4 User image David Zbarsky (:dzbarsky) 2013-01-09 15:05:09 PST
Comment 5 User image Ed Morley [:emorley] 2013-01-10 01:05:20 PST
Comment 6 User image Kohei Yoshino [:kohei] 2013-02-23 23:32:39 PST
I've added this bug to the compatibility doc. Please correct the info if I'm wrong.

Note You need to log in before you can comment on or make changes to this bug.