Web Animations: Don't compare start time for AnimationPlayer sorting

Source/core/animation/AnimationPlayer.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 44 matching lines @@
     player->suspendIfNeeded();
     return player.release();
 }
 
 AnimationPlayer::AnimationPlayer(ExecutionContext* executionContext, AnimationTimeline& timeline, AnimationNode* content)
     : ActiveDOMObject(executionContext)
     , m_playbackRate(1)
     , m_startTime(nullValue())
     , m_holdTime(nullValue())
     , m_storedTimeLag(0)
-    , m_sortInfo(nextSequenceNumber(), timeline.effectiveTime())
+    , m_sortInfo(nextSequenceNumber())
     , m_content(content)
     , m_timeline(&timeline)
     , m_paused(false)
     , m_held(false)
     , m_isPausedForTesting(false)
     , m_outdated(true)
     , m_finished(false)
 {
     ScriptWrappable::init(this);
     if (m_content) {
@@ skipping 100 matching lines @@
     ASSERT(!isUpdateFromCompositor || !hasStartTime());
 
     if (!std::isfinite(newStartTime))
         return;
     if (newStartTime == m_startTime)
         return;
     updateCurrentTimingState(); // Update the value of held
     bool hadStartTime = hasStartTime();
     double previousCurrentTime = currentTimeInternal();
     m_startTime = newStartTime;
-    m_sortInfo.m_startTime = newStartTime;
     updateCurrentTimingState();
     if (previousCurrentTime != currentTimeInternal()) {
         setOutdated();
     } else if (!hadStartTime && m_timeline) {
         // Even though this player is not outdated, time to effect change is
         // infinity until start time is set.
         m_timeline->wake();
     }
     if (!isUpdateFromCompositor) {
         cancelAnimationOnCompositor();
@@ skipping 211 matching lines @@
     if (m_playbackRate > 0)
         return m_content->timeToForwardsEffectChange() / m_playbackRate;
     return m_content->timeToReverseEffectChange() / -m_playbackRate;
 }
 
 void AnimationPlayer::cancel()
 {
     setSource(0);
 }
 
-bool AnimationPlayer::SortInfo::operator<(const SortInfo& other) const
-{
-    ASSERT(!std::isnan(m_startTime) && !std::isnan(other.m_startTime));
-    if (m_startTime < other.m_startTime)
-        return true;
-    if (m_startTime > other.m_startTime)
-        return false;
-    return m_sequenceNumber < other.m_sequenceNumber;
-}
-
 #if !ENABLE(OILPAN)
 bool AnimationPlayer::canFree() const
 {
     ASSERT(m_content);
     return hasOneRef() && m_content->isAnimation() && m_content->hasOneRef();
 }
 #endif
 
 bool AnimationPlayer::addEventListener(const AtomicString& eventType, PassRefPtr<EventListener> listener, bool useCapture)
 {
@@ skipping 14 matching lines @@
 
 void AnimationPlayer::trace(Visitor* visitor)
 {
     visitor->trace(m_content);
     visitor->trace(m_timeline);
     visitor->trace(m_pendingFinishedEvent);
     EventTargetWithInlineData::trace(visitor);
 }
 
 } // namespace