Bug 1046055 part 2 - Rename AnimationPlayer::IsCurrent to HasCurrentSource; r=dbaron

It's not the player that's "current" (a Web Animations term for an animation
that hasn't yet finished), but its source content, if any. This patch renames
the method on AnimationPlayer accordingly.

At the same time this patch moves the method to the header file since it's
quite simple and could possibly benefit from inlining.

content/base/src/Element.cpp

@@ -2958,7 +2958,7 @@ Element::GetAnimationPlayers(nsTArray<nsRefPtr<AnimationPlayer> >& aPlayers)
          playerIdx < collection->mPlayers.Length();
          playerIdx++) {
       AnimationPlayer* player = collection->mPlayers[playerIdx];
-      if (player->IsCurrent()) {
+      if (player->HasCurrentSource()) {
         aPlayers.AppendElement(player);
       }
     }

dom/animation/AnimationPlayer.cpp

@@ -64,12 +64,6 @@ AnimationPlayer::IsRunning() const
   return computedTiming.mPhase == ComputedTiming::AnimationPhase_Active;
 }
 
-bool
-AnimationPlayer::IsCurrent() const
-{
-  return GetSource() && GetSource()->IsCurrent();
-}
-
 Nullable<TimeDuration>
 AnimationPlayer::GetCurrentTimeDuration() const
 {

dom/animation/AnimationPlayer.h

@@ -64,7 +64,10 @@ public:
   }
 
   bool IsRunning() const;
-  bool IsCurrent() const;
+
+  bool HasCurrentSource() const {
+    return GetSource() && GetSource()->IsCurrent();
+  }
 
   // Return the duration since the start time of the player, taking into
   // account the pause state.  May be negative or null.

layout/style/AnimationCommon.cpp

@@ -722,7 +722,7 @@ bool
 AnimationPlayerCollection::HasCurrentAnimations()
 {
   for (size_t playerIdx = mPlayers.Length(); playerIdx-- != 0; ) {
-    if (mPlayers[playerIdx]->IsCurrent()) {
+    if (mPlayers[playerIdx]->HasCurrentSource()) {
       return true;
     }
   }