835030 - Remove svg.smil.enabled pref

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Robert Longson [:longsonr]]

Time for it to retire.

Comment 1

[Robert Longson [:longsonr]]

Attachment: patch

Comment 2

[Daniel Holbert [:dholbert]]

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

Looks good to me!

Comment 3

[Robert Longson [:longsonr]]

https://tbpl.mozilla.org/?tree=Try&rev=70d0db7b82e2

Comment 4

[Robert Longson [:longsonr]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/cb77bfbfbda6

Comment 5

[David Zbarsky (:dzbarsky)]

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

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

::: content/svg/content/src/SVGSVGElement.cpp
@@ +434,2 @@
>    }
> +  // else we're not the outermost <svg> or not bound to a tree, so silently fail

Since this function never throws rv, can you please remove the ErrorResult parameter and the [Throws] annotation in SVGSVGElement.webidl?

@@ +447,5 @@
>  void
>  SVGSVGElement::UnpauseAnimations(ErrorResult& rv)
>  {
> +  if (mTimedDocumentRoot) {
> +    mTimedDocumentRoot->Resume(nsSMILTimeContainer::PAUSE_SCRIPT);

Ditto

@@ +465,5 @@
>  bool
>  SVGSVGElement::AnimationsPaused(ErrorResult& rv)
>  {
> +  nsSMILTimeContainer* root = GetTimedDocumentRoot();
> +  return root && root->IsPausedByType(nsSMILTimeContainer::PAUSE_SCRIPT);

Ditto

@@ +485,5 @@
> +  if (root) {
> +    double fCurrentTimeMs = double(root->GetCurrentTime());
> +    return (float)(fCurrentTimeMs / PR_MSEC_PER_SEC);
> +  } else {
> +    return 0.f;

Ditto

@@ +516,5 @@
> +    //  - Make sure we get an up-to-date paint after this method
> +    //  - re-enable event firing (it got disabled during seeking, and it
> +    //  doesn't get re-enabled until the first sample after the seek -- so
> +    //  let's make that happen now.)
> +    FlushAnimations();

Ditto

Comment 6

[Robert Longson [:longsonr]]

Attachment: followup

Comment 7

[Robert Longson [:longsonr]]

Attachment: patch

Comment 8

[David Zbarsky (:dzbarsky)]

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

Thanks! r=me

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/cb77bfbfbda6

Comment 10

[Robert Longson [:longsonr]]

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

Doesn't compile

Comment 11

[Robert Longson [:longsonr]]

Changing the signatures is complicated by the nsIDOMSVGVSGElement interface. I think it's going to need to be a follow up.

Comment 12

[David Zbarsky (:dzbarsky)]

OK, I'll do it as part of bug 833446

Comment 13

[Jean-Yves Perrier [:teoli]]

As there was no mention of svg.smil.enabled in the reference, I only added a note in 
https://developer.mozilla.org/en-US/Firefox/Releases/21#SVG