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

Unified Diff: third_party/WebKit/Source/core/animation/ComputedTimingProperties.idl

Issue 1944603003: Make AnimationEffectReadOnly.getComputedTiming() consistent with Web Animations spec (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 4 years, 8 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: third_party/WebKit/Source/core/animation/ComputedTimingProperties.idl
diff --git a/third_party/WebKit/Source/core/animation/ComputedTimingProperties.idl b/third_party/WebKit/Source/core/animation/ComputedTimingProperties.idl
index 71e053007648d6abb3af2e5ad6dffbb65eea9ca5..1df661e046f575e860bcb44c75b3c970dfdbc476 100644
--- a/third_party/WebKit/Source/core/animation/ComputedTimingProperties.idl
+++ b/third_party/WebKit/Source/core/animation/ComputedTimingProperties.idl
@@ -5,13 +5,9 @@
// https://w3c.github.io/web-animations/#the-computedtimingproperties-dictionary
dictionary ComputedTimingProperties : KeyframeEffectOptions {
- // TODO(dstockwell): There is no startTime member in the spec.
- double startTime;
- unrestricted double endTime;
- unrestricted double activeDuration;
- double? localTime;
- // TODO(dstockwell): The timeFraction and currentIteration members should
- // both be of type unrestricted double? (nullable).
- unrestricted double timeFraction;
- unsigned long? currentIteration;
+ unrestricted double endTime;
+ unrestricted double activeDuration;
+ double? localTime;
+ double? progress;
+ unrestricted double? currentIteration;
};

Powered by Google App Engine
This is Rietveld 408576698