Closed Bug 1266273 Opened 3 years ago Closed 3 years ago
Change type of Computed
Timing Properties .progress from unrestricted double? to double?
The spec has changed the type of this dictionary because it doesn't seem like there are any cases where this will be +/-Infinity or NaN: https://github.com/w3c/web-animations/commit/df138ee1314da46c6d671d7e406433c3e675ee61 Bug 1266257 adds an assertion that the progress is finite before we set it (specifically, the very end of the first patch in that series) and it doesn't seem to have failed in any of our automated tests so it seems like this change is ok.
To reflect updates to the spec: https://github.com/w3c/web-animations/commit/df138ee1314da46c6d671d7e406433c3e675ee61 MozReview-Commit-ID: 525eLGCK4tl
Attachment #8743642 - Flags: review?(bzbarsky)
Comment on attachment 8743642 [details] [diff] [review] Change type of ComputedTimingProperties.progress from unrestricted double? to double? r=me
Attachment #8743642 - Flags: review?(bzbarsky) → review+
You need to log in before you can comment on or make changes to this bug.