1340086 - Remove uses of AnimationsPaused() since it is always false after bug 1316330

Status: RESOLVED FIXED

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

Description

[Jessica Jong [:jessica] (inactive)]

Follow-up to bug 1316330.

Bug 1316330 drops the use of suppressing eAnimationsOnly, so I think we can remove AnimationsPaused().

Comment 1

[Jessica Jong [:jessica] (inactive)]

Attachment: patch, v1.

Comment 2

[Jessica Jong [:jessica] (inactive)]

Comment on attachment 8838446 [details] [diff] [review]
patch, v1.

Hi Blake, may I have your review on this? Thanks.

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 8838446 [details] [diff] [review]
patch, v1.

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

Thanks.

::: dom/base/nsDocument.cpp
@@ +9679,5 @@
>  
>  void
> +nsDocument::UnsuppressEventHandlingAndFireEvents(bool aFireEvents)
> +{
> +  nsTArray<nsCOMPtr<nsIDocument> > documents;

Might as well remove the now-unnecessary space between the >s (nsTArray<nsCOMPtr<nsIDocument>>).

Comment 4

[Jessica Jong [:jessica] (inactive)]

Attachment: patch, v2.

Addressed nit in comment 3, carrying r+.

Comment 5

[Jessica Jong [:jessica] (inactive)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e8368750808e0a6a666618e18a02f3d0794cd7a9&group_state=expanded

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/20c29c6769fe
Remove uses of AnimationsPaused() since it is always false after bug 1316330. r=mrbkap

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/20c29c6769fe