Declare onFinish event handler for AnimationPlayer

Source/core/animation/DocumentTimeline.cpp

Index: Source/core/animation/DocumentTimeline.cpp
diff --git a/Source/core/animation/DocumentTimeline.cpp b/Source/core/animation/DocumentTimeline.cpp
index 77ef592d566ff005da38639a7d62257444a653db..1471c1f6943cba09623f7f325c64f9ad5d53f8fb 100644
--- a/Source/core/animation/DocumentTimeline.cpp
+++ b/Source/core/animation/DocumentTimeline.cpp
@@ -70,7 +70,7 @@ DocumentTimeline::~DocumentTimeline()
 
 AnimationPlayer* DocumentTimeline::createAnimationPlayer(TimedItem* child)
 {
-    RefPtr<AnimationPlayer> player = AnimationPlayer::create(*this, child);
+    RefPtr<AnimationPlayer> player = AnimationPlayer::create(m_document ? m_document->contextDocument().get() : 0, *this, child);

[alancutter (OOO until 2018), 2014/03/31 06:28:13]

Ideally a JS handle to DocumentTimeline would keep Document alive and this check
for null would not be needed. Perhaps we should have an exit if null check in
play() with a FIXME for now to make intended behaviour more clear.

[Eric Willigers, 2014/03/31 08:48:47]

Done.

     AnimationPlayer* result = player.get();
     m_players.add(result);
     setOutdatedAnimationPlayer(result);