Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2098)

Unified Diff: Source/devtools/front_end/elements/AnimationControlPane.js

Issue 1156323003: Devtools Animations: Update timeline controls and UI (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/devtools/front_end/elements/AnimationControlPane.js
diff --git a/Source/devtools/front_end/elements/AnimationControlPane.js b/Source/devtools/front_end/elements/AnimationControlPane.js
index 7f1eddb73e0b6d966bbd63e29d07a3bafd9a5428..b0604bc6859625d30dd2d79010b444321226a1aa 100644
--- a/Source/devtools/front_end/elements/AnimationControlPane.js
+++ b/Source/devtools/front_end/elements/AnimationControlPane.js
@@ -135,9 +135,14 @@ WebInspector.AnimationControlPane.ButtonProvider.prototype = {
this._animationControlPaneModeClick();
},
- _nodeChanged: function()
+ /**
+ * @param {!Event=} event
+ */
+ _nodeChanged: function(event)
{
this._button.setEnabled(!!WebInspector.context.flavor(WebInspector.DOMNode));
+ if (event && this._animationTimeline)
pfeldman 2015/06/05 14:34:34 When is event undefined?
samli 2015/06/09 03:53:08 Called in WebInspector.AnimationControlPane.Button
+ this._animationTimeline.setNode(/** @type {!WebInspector.DOMNode} */(event.data));
},
/**

Powered by Google App Engine
This is Rietveld 408576698