1521946 - Move nsSMILTypes and nsSMILValue to the mozilla namespace

Status: RESOLVED FIXED

(Core :: SVG, enhancement, P3)

Comment 1

[Robert Longson [:longsonr]]

Attachment: 1521946.txt

Comment 2

[Brian Birtles (:birtles)]

Comment on attachment 9038364 [details] [diff] [review]
1521946.txt

Review of attachment 9038364 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/smil/SMILCSSValueType.cpp
@@ +16,3 @@
>  #include "nsPresContext.h"
> +#include "nsString.h"
> +#include "nsStyleUtil.h"

Thanks for fixing this ordering!

::: dom/smil/SMILCSSValueType.h
@@ +84,5 @@
>     * @param aPropID         The property that |aValue| corresponds to.
>     * @param aTargetElement  The target element to which the animation value
>     *                        applies.
>     * @param aValue          The animation value to use.
> +   * @return                A new SMILValue. On failure, returns an

Super minor nit: 'a SMILValue'.

Applies many other places in this patch but it's also probably not worth fixing unless you feel so inclined.

::: dom/svg/SVGAngle.cpp
@@ +11,1 @@
>  #include "mozilla/Move.h"

Nit: Order here

::: dom/svg/SVGAnimatedLengthList.cpp
@@ +13,2 @@
>  #include "nsSVGAttrTearoffTable.h"
> +#include "DOMSVGAnimatedLengthList.h"

I probably would have expected 'D' to come before 'n' unless you're doing a case-sensitive sort.

::: dom/svg/SVGAnimatedPathSegList.cpp
@@ +13,1 @@
>  #include "DOMSVGPathSegList.h"

Likewise here.

::: dom/svg/SVGAnimatedPreserveAspectRatio.cpp
@@ +14,1 @@
>  #include "nsSVGAttrTearoffTable.h"

Similarly here.

::: dom/svg/SVGBoolean.cpp
@@ +11,1 @@
>  #include "nsSVGAttrTearoffTable.h"

Ordering here seems odd.

::: dom/svg/SVGEnum.cpp
@@ +14,1 @@
>  #include "nsSVGAttrTearoffTable.h"

Here too (or is the idea to sort ignoring the 'ns'?).

Comment 3

[Robert Longson [:longsonr]]

The idea is to ignore the ns because it's on my to-do list to fix that and it's just making work to keep reordering that.

I was also ignoring DOM so that DOMSVGAngle went next to SVGAngle. DOMSVGAngle is the DOM tear off for SVGAngle.

Comment 4

[Robert Longson [:longsonr]]

I've fixed some of the ordering, the rest can wait till I do nsSVGAttrTearoffTable

I've also had a go (via sed) at replacing an SMILValue by a SMILValue.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/551abefb4933
Move nsSMILTypes and nsSMILValue to the mozilla namespace r=birtles

Comment 6

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/551abefb4933