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

Unified Diff: Source/devtools/front_end/animation/AnimationTimeline.js

Issue 1324193003: Devtools Animations: Update only page targets (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/animation/AnimationTimeline.js
diff --git a/Source/devtools/front_end/animation/AnimationTimeline.js b/Source/devtools/front_end/animation/AnimationTimeline.js
index a619f6cc9d5d6f8b289b42027a9d503df80fd6a8..1556f3f95a8d727ebf87253c37022fc7356ed060 100644
--- a/Source/devtools/front_end/animation/AnimationTimeline.js
+++ b/Source/devtools/front_end/animation/AnimationTimeline.js
@@ -206,9 +206,9 @@ WebInspector.AnimationTimeline.prototype = {
{
if (this.startTime() === undefined)
return;
- var targets = WebInspector.targetManager.targets();
- for (var target of targets)
+ for (var target of WebInspector.targetManager.targets(WebInspector.Target.Type.Page))
target.animationAgent().setCurrentTime(/** @type {number} */(this.startTime()));
+
this._animateTime(0);
},
@@ -507,8 +507,7 @@ WebInspector.AnimationTimeline.prototype = {
this._scrubberPlayer.currentTime = Math.min(this._originalScrubberTime + delta / this.pixelMsRatio(), this.duration() - this._scrubberRadius / this.pixelMsRatio());
var currentTime = Math.max(0, Math.round(this._scrubberPlayer.currentTime));
this._timelineScrubberHead.textContent = WebInspector.UIString(Number.millisToString(currentTime));
- var targets = WebInspector.targetManager.targets();
- for (var target of targets)
+ for (var target of WebInspector.targetManager.targets(WebInspector.Target.Type.Page))
target.animationAgent().setCurrentTime(/** @type {number} */(this.startTime() + currentTime));
},
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698