Devtools Animations: Update transition timing on timeline interaction

Source/devtools/front_end/elements/AnimationTimeline.js

 // Copyright (c) 2015 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.
 
 /**
  * @constructor
  * @extends {WebInspector.VBox}
  */
 WebInspector.AnimationTimeline = function()
 {
@@ skipping 768 matching lines @@
         this._movementInMs = (event.clientX - this._downMouseX) / this._timeline.pixelMsRatio();
 
         // Commit changes
         if (this._mouseEventType === WebInspector.AnimationUI.MouseEvents.KeyframeMove) {
             this._keyframes[this._keyframeMoved].setOffset(this._offset(this._keyframeMoved));
         } else {
             var delay = this._delay();
             var duration = this._duration();
             this._setDelay(delay);
             this._setDuration(duration);
-            // FIXME: Transition timing updates currently not supported
-            if (this._animation.type() == "WebAnimation") {
+            if (this._animation.type() !== "CSSAnimation") {

[dgozman, 2015/03/30 10:38:58]

What about CSSAnimation? Should you leave the FIXME?

[samli, 2015/03/30 23:14:17]

No, CSS animations are already supported in http://crrev.com/895783004. The
inline style is updated and changes are automatically detected by the animations
engine. Transitions are not spec-ed in the same manner which is why we need to
update it in this manner.

                 var target = WebInspector.targetManager.mainTarget();
                 if (target)
-                    target.animationAgent().setTiming(this._animation.id(), duration, delay);
+                    target.animationAgent().setTiming(this._animation.id(), duration, delay, this._animation.type() === "CSSTransition");
             }
         }
 
         this._movementInMs = 0;
         this.redraw();
 
         this._parentElement.ownerDocument.removeEventListener("mousemove", this._mouseMoveHandler);
         this._parentElement.ownerDocument.removeEventListener("mouseup", this._mouseUpHandler);
         delete this._mouseMoveHandler;
         delete this._mouseUpHandler;
@@ skipping 92 matching lines @@
     "Light Blue": WebInspector.Color.parse("#03A9F4"),
     "Deep Orange": WebInspector.Color.parse("#FF5722"),
     "Blue": WebInspector.Color.parse("#5677FC"),
     "Lime": WebInspector.Color.parse("#CDDC39"),
     "Blue Grey": WebInspector.Color.parse("#607D8B"),
     "Pink": WebInspector.Color.parse("#E91E63"),
     "Green": WebInspector.Color.parse("#0F9D58"),
     "Brown": WebInspector.Color.parse("#795548"),
     "Cyan": WebInspector.Color.parse("#00BCD4")
 }