Closed
Bug 824218
Opened 11 years ago
Closed 11 years ago
Remove unimplemented SVG features
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: dzbarsky, Assigned: dzbarsky)
Details
Attachments
(1 file)
29.84 KB,
patch
|
longsonr
:
review+
|
Details | Diff | Splinter Review |
It doesn't do anything useful.
Assignee | ||
Comment 1•11 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•11 years ago
|
||
Attachment #698529 -
Flags: review?(longsonr)
Comment 3•11 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•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9297bf41ab4d
Comment 5•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9297bf41ab4d
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Comment 6•11 years ago
|
||
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.
Description
•