1522650 - Complete migration of SMIL to the mozilla namespace

Status: RESOLVED FIXED

(Core :: SVG, enhancement)

Comment 1

[Robert Longson [:longsonr]]

Attachment: patch

Comment 2

[Brian Birtles (:birtles, Feb 26~ parental leave)]

Comment on attachment 9038914 [details] [diff] [review]
patch

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

Thanks for doing this. It looks great (thanks for taking care of the comments, dropping typedefs etc.).

Regarding the updated include guards, feel free to do that in a separate patch/bug if you prefer.

::: dom/base/Element.h
@@ +18,5 @@
> +#include "nsAttrValueInlines.h"
> +#include "nsChangeHint.h"
> +#include "nsContentUtils.h"
> +#include "nsDOMAttributeMap.h"
> +#include "DOMIntersectionObserver.h"

(I'm a little surprised by the "sort by ignoring the 'ns' prefix" approach, but if that's what you and dholbert discussed, or that's what you're seeing elsewhere, it's fine with me.)

::: dom/smil/SMILAttr.h
@@ +100,1 @@
>  #endif  // NS_ISMILATTR_H_

I forgot to check this in the other patches in this series, but I think we need to update these include guards to use:

  mozilla_SMILAttr_h

etc.

[1] https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style#CC_practices

::: dom/svg/SVGAnimationElement.h
@@ -21,5 @@
> -enum nsSMILTargetAttrType {
> -  eSMILTargetAttrType_auto,
> -  eSMILTargetAttrType_CSS,
> -  eSMILTargetAttrType_XML
> -};

Nice catch!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9c985b12ab62
Rename all remaining nsSMIL classes and types as SMIL and ensure they are in the mozilla namespace. r=birtles

Comment 4

[Robert Longson [:longsonr]]

I left the include headers for another bug, I did make the ordering in Element.h look less odd though.

Comment 5

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/9c985b12ab62