Declare onFinish event handler for AnimationPlayer

Source/core/animation/DocumentTimeline.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 52 matching lines @@
 }
 
 DocumentTimeline::~DocumentTimeline()
 {
     for (HashSet<AnimationPlayer*>::iterator it = m_players.begin(); it != m_players.end(); ++it)
         (*it)->timelineDestroyed();
 }
 
 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);
     return result;
 }
 
 AnimationPlayer* DocumentTimeline::play(TimedItem* child)
 {
     AnimationPlayer* player = createAnimationPlayer(child);
     player->setStartTime(effectiveTime());
@@ skipping 103 matching lines @@
             count += (timedItem && (timedItem->isCurrent() || timedItem->isInEffect()));
     }
     return count;
 }
 
 void DocumentTimeline::detachFromDocument() {
     m_document = 0;
 }
 
 } // namespace