610216 - Build bustage when SMIL is disabled

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Mounir Lamouri (:mounir)]

Attachment: Patch v1

Comment 1

[Daniel Holbert [:dholbert]]

Comment on attachment 488746 [details] [diff] [review]
Patch v1

Could you narrow the #ifdefs to *just* be around these single-lines:

     doc->GetAnimationController()->Resume(nsSMILTimeContainer::PAUSE_IMAGE);
and
     doc->GetAnimationController()->Pause(nsSMILTimeContainer::PAUSE_IMAGE);

Technically, we need the other stuff (SetImagesNeedAnimating) in non-SMIL builds, because it controls animated GIFs / PNGs, too.

r=dholbert with that.  Thanks for the fix, and sorry for busting your build! :)

Comment 2

[Mounir Lamouri (:mounir)]

Thanks for your quick review.

Pushed:
http://hg.mozilla.org/mozilla-central/rev/c6b6c4d8b9bb