Devtools Animations: Update transition timing on timeline interaction

Source/core/inspector/InspectorAnimationAgent.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 
 #include "core/inspector/InspectorAnimationAgent.h"
 
 #include "core/animation/Animation.h"
 #include "core/animation/AnimationEffect.h"
@@ skipping 228 matching lines @@
         if (frame->isLocalFrame())
             toLocalFrame(frame)->document()->timeline().setPlaybackRate(playbackRate);
     }
 }
 
 void InspectorAnimationAgent::setCurrentTime(ErrorString*, double currentTime)
 {
     m_pageAgent->inspectedFrame()->document()->timeline().setCurrentTime(currentTime);
 }
 
-void InspectorAnimationAgent::setTiming(ErrorString* errorString, const String& playerId, double duration, double delay)
+void InspectorAnimationAgent::setTiming(ErrorString* errorString, const String& playerId, double duration, double delay, bool isTransition)
 {
     AnimationPlayer* player = assertAnimationPlayer(errorString, playerId);
     if (!player)
         return;
 
-    RefPtrWillBeRawPtr<AnimationNodeTiming> timing = player->source()->timing();
-    UnrestrictedDoubleOrString unrestrictedDuration;
-    unrestrictedDuration.setUnrestrictedDouble(duration);
-    timing->setDuration(unrestrictedDuration);
-    timing->setDelay(delay);
+    if (isTransition) {
+        Animation* animation = toAnimation(player->source());
+        KeyframeEffectModelBase* effect = toKeyframeEffectModelBase(animation->effect());
+        ASSERT(effect->isAnimatableValueKeyframeEffectModel());
+        const AnimatableValueKeyframeEffectModel* oldEffect = toAnimatableValueKeyframeEffectModel(effect);
+        const KeyframeVector& frames = oldEffect->getFrames();
+
+        KeyframeVector newFrames;
+        // Copy frames
+        for (int i = 0; i < 3; i++)

[dgozman, 2015/03/26 10:30:05]

Why exactly 3 elements?

[dstockwell, 2015/03/26 22:56:16]

ASSERT that there are exactly 3 keyframes

[samli, 2015/03/30 00:22:55]

dstockwell: Done.
dgozman: Our internal representation of transitions now uses 3 keyframes all the
time.

+            newFrames.append(toAnimatableValueKeyframe(frames[i]->clone().get()));
+        // Update delay

[dstockwell, 2015/03/26 22:56:16]

// Update delay, represented by the distance between the first two keyframes.

[samli, 2015/03/30 00:22:55]

Done.

+        newFrames[1]->setOffset(delay / (delay + duration));

[dgozman, 2015/03/26 10:30:05]

Why only first frame?

[samli, 2015/03/30 00:22:55]

As per updated comment, transition delays are represented by the offset of the
second frame.

+        effect->setFrames(newFrames);
+        // Update duration

[dgozman, 2015/03/26 10:30:05]

These comments seem unnecessary.

[samli, 2015/03/30 00:22:55]

Done.

+        RefPtrWillBeRawPtr<AnimationNodeTiming> timing = player->source()->timing();
+        UnrestrictedDoubleOrString unrestrictedDuration;
+        unrestrictedDuration.setUnrestrictedDouble(duration + delay);
+        timing->setDuration(unrestrictedDuration);
+    } else {
+        RefPtrWillBeRawPtr<AnimationNodeTiming> timing = player->source()->timing();
+        UnrestrictedDoubleOrString unrestrictedDuration;
+        unrestrictedDuration.setUnrestrictedDouble(duration);
+        timing->setDuration(unrestrictedDuration);
+        timing->setDelay(delay);
+    }
 }
 
 void InspectorAnimationAgent::didCreateAnimationPlayer(AnimationPlayer* player)
 {
     const String& playerId = String::number(player->sequenceNumber());
     if (m_idToAnimationPlayer.get(playerId))
         return;
 
     // Check threshold
     double latestStartTime = 0;
@@ skipping 32 matching lines @@
 {
 #if ENABLE(OILPAN)
     visitor->trace(m_pageAgent);
     visitor->trace(m_domAgent);
     visitor->trace(m_idToAnimationPlayer);
 #endif
     InspectorBaseAgent::trace(visitor);
 }
 
 }