Make the timer in SMILTimeContainer more of an implementation detail

Make the timer in SMILTimeContainer more of an implementation detail

Add a few new methods to represent the different actions required:

 scheduleAnimationFrame(SMILTime fireTime)
 immediateAnimationFrame()
 cancelAnimationFrame()

and replace uses of m_timer.foo() and startTimer with the appropriate
method.

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

[pdr., 2014-02-04 18:45:57]

Another fantastic patch, thank you!

LGTM with one nit.

https://codereview.chromium.org/143403005/diff/1/Source/core/svg/animation/SM...
File Source/core/svg/animation/SMILTimeContainer.cpp (right):

https://codereview.chromium.org/143403005/diff/1/Source/core/svg/animation/SM...
Source/core/svg/animation/SMILTimeContainer.cpp:223: void
SMILTimeContainer::immediateAnimationFrame()
Nit: can you rename this scheduleAnimationFrame? This is immediate in the sense
that it immediately schedules the timer, but the timer itself is still an
asynchronous event.

[fs, 2014-02-05 12:43:21]

On 2014/02/04 18:45:57, pdr wrote:
> Another fantastic patch, thank you!
> 
> LGTM with one nit.
> 
>
https://codereview.chromium.org/143403005/diff/1/Source/core/svg/animation/SM...
> File Source/core/svg/animation/SMILTimeContainer.cpp (right):
> 
>
https://codereview.chromium.org/143403005/diff/1/Source/core/svg/animation/SM...
> Source/core/svg/animation/SMILTimeContainer.cpp:223: void
> SMILTimeContainer::immediateAnimationFrame()
> Nit: can you rename this scheduleAnimationFrame? This is immediate in the
sense
> that it immediately schedules the timer, but the timer itself is still an
> asynchronous event.

Done.

[fs, 2014-02-05 12:43:30]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2014-02-05 12:43:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fs@opera.com/143403005/90001

[commit-bot: I haz the power, 2014-02-05 13:57:47]

Change committed as 166497