Bug 46945 - Make suspendAnimations/resumeAnimations and setCSSAnimations traverse through subframes and remember state
: Make suspendAnimations/resumeAnimations and setCSSAnimations traverse through...
Status: RESOLVED FIXED
: WebKit
Layout and Rendering
: 528+ (Nightly build)
: PC Mac OS X 10.5
: P2 Normal
Assigned To:
:
: InRadar
:
: 46525 46540
  Show dependency treegraph
 
Reported: 2010-09-30 17:26 PST by
Modified: 2010-11-05 09:56 PST (History)


Attachments
Patch (18.69 KB, patch)
2010-11-04 15:56 PST, Chris Marrin
simon.fraser: review+
Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2010-09-30 17:26:47 PST
Currently suspendAnimations/resumeAnimations in WebCore take a document, but the pass through from WebKit, which allows access from LayoutTestController simply passes in the top-level document. The WebKit versions should traverse all subframes. Also, we don't remember the suspend state in AnimationController, so starting a new animation while suspended doesn't suspend.

I'm fixing these together because the test case I am writing needs the functionality in the former to test the latter.
------- Comment #1 From 2010-09-30 17:32:55 PST -------
Dup of bug 46429?
------- Comment #2 From 2010-10-01 08:33:11 PST -------
*** Bug 46944 has been marked as a duplicate of this bug. ***
------- Comment #3 From 2010-10-01 08:34:59 PST -------
*** Bug 46429 has been marked as a duplicate of this bug. ***
------- Comment #4 From 2010-10-01 08:35:45 PST -------
<rdar://problem/8472265>
------- Comment #5 From 2010-11-04 15:56:22 PST -------
Created an attachment (id=72995) [details]
Patch
------- Comment #6 From 2010-11-04 16:17:14 PST -------
(From update of attachment 72995 [details])
View in context: https://bugs.webkit.org/attachment.cgi?id=72995&action=review

> LayoutTests/animations/animation-test-helpers.js:82
> +        }
> +    }

inIframe is redundant, because you could just test if (iframeId)...

> LayoutTests/animations/stop-animation-on-suspend.html:56
> +        setTimeout(suspend, 250);
> +        setTimeout(resume, 500);

Can we make this test faster?
------- Comment #7 From 2010-11-05 09:51:03 PST -------
(From update of attachment 72995 [details])
We should not be adding new functions to Frame. We’ve been working hard slowly moving functions off of it and adding new ones is not a good idea. These function should go on the animation controller.
------- Comment #8 From 2010-11-05 09:55:35 PST -------
(In reply to comment #7)
> (From update of attachment 72995 [details] [details])
> We should not be adding new functions to Frame. We’ve been working hard slowly moving functions off of it and adding new ones is not a good idea. These function should go on the animation controller.

Sorry, but I already checked in before seeing this. I have opened a new bug (https://bugs.webkit.org/show_bug.cgi?id=49073) to make this change.
------- Comment #9 From 2010-11-05 09:56:16 PST -------
Landed in http://trac.webkit.org/changeset/71424