825730 - Various SVG cleanups

Status: RESOLVED FIXED

(Core :: SVG, defect)

Comment 1

[David Zbarsky (:dzbarsky)]

Attachment: Alphabetize makefile

Comment 2

[David Zbarsky (:dzbarsky)]

Attachment: Add a more convenient signature for ToDOMAnimatedLength

Comment 3

[David Zbarsky (:dzbarsky)]

Attachment: Include headers as mozilla/dom

Comment 4

[Robert Longson [:longsonr]]

Comment on attachment 696860 [details] [diff] [review]
Add a more convenient signature for ToDOMAnimatedLength

> {
>-  nsCOMPtr<nsIDOMSVGAnimatedLength> cx;
>-  mLengthAttributes[CX].ToDOMAnimatedLength(getter_AddRefs(cx), this);
>-  return cx.forget();
>+  return mLengthAttributes[ATTR_CX].ToDOMAnimatedLength(this);

Does this compile? What's changing CX to ATTR_CX?

This occurs 2 or 3 other times

r=longsonr if it compiles.

Comment 5

[Robert Longson [:longsonr]]

Comment on attachment 696861 [details] [diff] [review]
Include headers as mozilla/dom

I assume this depends on other patches elsewhere. Do make sure it compiles when it lands.

Comment 6

[David Zbarsky (:dzbarsky)]

(In reply to Robert Longson from comment #4)
> Comment on attachment 696860 [details] [diff] [review]
> Add a more convenient signature for ToDOMAnimatedLength
> 
> > {
> >-  nsCOMPtr<nsIDOMSVGAnimatedLength> cx;
> >-  mLengthAttributes[CX].ToDOMAnimatedLength(getter_AddRefs(cx), this);
> >-  return cx.forget();
> >+  return mLengthAttributes[ATTR_CX].ToDOMAnimatedLength(this);
> 
> Does this compile? What's changing CX to ATTR_CX?
> 
> This occurs 2 or 3 other times
> 
> r=longsonr if it compiles.

I changed those when adding the WebIDL bindings because having X be both in the enum and a function doesn't compile too well.  These patches compile but they're interspersed with other patches in my queue so I'll land them together.

Comment 7

[David Zbarsky (:dzbarsky)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2a2a7a9d4d3e
https://hg.mozilla.org/integration/mozilla-inbound/rev/04e417ee3fe8

Comment 8

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/2a2a7a9d4d3e
https://hg.mozilla.org/mozilla-central/rev/04e417ee3fe8