Web Animations: Remove AnimationPlayer discarding logic

Web Animations: Remove AnimationPlayer discarding logic

Since forwards filling animations are supposed to be returned via
getAnimationPlayers, we can no longer discard them when we've dropped
references to them. This patch removes this discarding logic.

BUG=439247
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=190977

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=191085

[Timothy Loh, 2015-02-26 05:22:57]

timloh@chromium.org changed reviewers:
+ dstockwell@chromium.org, samli@chromium.org

[Timothy Loh, 2015-02-27 00:40:47]

Should work now

[samli, 2015-02-27 00:53:37]

lgtm

[Timothy Loh, 2015-02-27 02:21:35]

The CQ bit was checked by timloh@chromium.org

[commit-bot: I haz the power, 2015-02-27 02:22:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/955303002/20001

[commit-bot: I haz the power, 2015-02-27 05:01:44]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=190977

[Mike West, 2015-02-27 10:46:11]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/967523002/ by mkwst@chromium.org.

The reason for reverting is: It looks like this patch broke `webkit_unit_tests`
(`WebFrameSwapTest.SwapPreservesGlobalContext` in particular) on a number of
platforms.
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux/builds/4...
for example..

[Timothy Loh, 2015-03-02 04:03:35]

On 2015/02/27 10:46:11, Mike West wrote:
> A revert of this CL (patchset #2 id:20001) has been created in
> https://codereview.chromium.org/967523002/ by mailto:mkwst@chromium.org.
> 
> The reason for reverting is: It looks like this patch broke
`webkit_unit_tests`
> (`WebFrameSwapTest.SwapPreservesGlobalContext` in particular) on a number of
> platforms.
>
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux/builds/4...
> for example..

What other platforms were breaking, I had a look at the other builders and it
looked like it was only "WebKit Linux". I can't work out what's different about
it to the other builders/trybots :\

[Timothy Loh, 2015-03-02 05:11:01]

The CQ bit was checked by timloh@chromium.org

[Timothy Loh, 2015-03-02 05:11:02]

The patchset sent to the CQ was uploaded after l-g-t-m from samli@chromium.org
Link to the patchset: https://codereview.chromium.org/955303002/#ps40001 (title:
"don't actually delete unit test")

[commit-bot: I haz the power, 2015-03-02 05:11:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/955303002/40001

[commit-bot: I haz the power, 2015-03-02 08:05:28]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191085