Remove svg.smil.enabled pref

RESOLVED FIXED in mozilla21

Status

()

defect
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: longsonr, Assigned: longsonr)

Tracking

({dev-doc-complete})

Trunk
mozilla21
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

Assignee

Description

6 years ago
Time for it to retire.
Assignee

Comment 1

6 years ago
Posted patch patchSplinter Review
Assignee: nobody → longsonr
Attachment #706769 - Flags: review?(dholbert)
Comment on attachment 706769 [details] [diff] [review]
patch

Looks good to me!
Attachment #706769 - Flags: review?(dholbert) → review+
Status: NEW → ASSIGNED
Assignee

Updated

6 years ago
Keywords: dev-doc-needed
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
Assignee

Comment 6

6 years ago
Posted patch followup (obsolete) — Splinter Review
Attachment #706905 - Flags: review?(dzbarsky)
Assignee

Updated

6 years ago
Attachment #706905 - Attachment is obsolete: true
Attachment #706905 - Flags: review?(dzbarsky)
Assignee

Comment 7

6 years ago
Posted patch patch (obsolete) — Splinter Review
Attachment #706906 - Flags: review?(dzbarsky)
Assignee

Updated

6 years ago
Whiteboard: [leave open]
Comment on attachment 706906 [details] [diff] [review]
patch

Thanks! r=me
Attachment #706906 - Flags: review?(dzbarsky) → review+
Assignee

Comment 10

6 years ago
Comment on attachment 706906 [details] [diff] [review]
patch

Doesn't compile
Attachment #706906 - Attachment is obsolete: true
Assignee

Updated

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [leave open]
Assignee

Comment 11

6 years ago
Changing the signatures is complicated by the nsIDOMSVGVSGElement interface. I think it's going to need to be a follow up.
OK, I'll do it as part of bug 833446
Assignee

Updated

6 years ago
Target Milestone: --- → mozilla21
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
You need to log in before you can comment on or make changes to this bug.