Bug 1104427 part 1 - Rename GetStartTime to GetStartTimeAsDouble; r=bholley
commitfd2b033a2922566afe0eb0737ad6d3dba9bb1996
authorBrian Birtles <birtles@gmail.com>
Thu, 4 Dec 2014 16:28:38 +0000 (4 08:28 -0800)
committerBrian Birtles <birtles@gmail.com>
Thu, 4 Dec 2014 16:28:38 +0000 (4 08:28 -0800)
treed3d4c351bca3da133f7f8fdec1ab2267576c7e70
parent314850976802413e0e1a21bcba7e0f387616a3c7
Bug 1104427 part 1 - Rename GetStartTime to GetStartTimeAsDouble; r=bholley

In forthcoming patches we will encapsulate AnimationPlayer::mStartTime so we can
ensure that related state is updated appropriately. We would like to expose
mStartTime via GetStartTime() but currently a method of that name returns the
start time as a double.

This patch applies the pattern used for currentTime to startTime; specifically,
GetCurrentTime() returns the TimeDuration (since that's what C++ callers should
use) while GetCurrentTimeAsDouble() returns a double.

At the same time, this patch also removes the [Pure] extended attribute from
startTime in the WebIDL definition since subsequent patches either in this bug
or in bug 927349 will mean that startTime can be updated out-of-band.
Specifically, we will implement deferred playback of animation such that the
startTime remains null until we've finished rendering the first frame of the
animation.
dom/animation/AnimationPlayer.cpp
dom/animation/AnimationPlayer.h
dom/webidl/AnimationPlayer.webidl