Plumb timeToNextEffect through players and animation

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 56 matching lines @@
         TRACE_EVENT0("webkit", "DocumentTimeline::serviceAnimations");
         // FIXME: The below ASSERT fires on Windows when running chrome.exe.
         // Does not fire with --single-process, or on Linux, or in
         // content_shell.exe. The assert condition has been moved up into the
         // outer 'if' to work around this. http://crbug.com/280439.
         if (!isNull(m_zeroTimeAsPerfTime) && (m_currentTime <= monotonicAnimationStartTime - m_zeroTimeAsPerfTime)) {
             ASSERT(m_currentTime <= monotonicAnimationStartTime - m_zeroTimeAsPerfTime);
             m_currentTime = monotonicAnimationStartTime - m_zeroTimeAsPerfTime;
         }
 
+        double timeToNextEffect = -1;
         for (int i = m_players.size() - 1; i >= 0; --i) {
-            if (!m_players[i]->update())
+            if (!m_players[i]->update(&timeToNextEffect))

[dstockwell, 2013/10/21 01:51:31]

We could just not pass a param here. But I guess this will be fixed in the next
change.

                 m_players.remove(i);
         }
 
         if (m_document->view() && !m_players.isEmpty())
             m_document->view()->scheduleAnimation();
     }
 
     dispatchEvents();
 }
 
@@ skipping 22 matching lines @@
 }
 
 size_t DocumentTimeline::numberOfActiveAnimationsForTesting() const
 {
     // Includes all players whose directly associated timed items
     // are current or in effect.
     return isNull(m_currentTime) ? 0 : m_players.size();
 }
 
 } // namespace