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

Unified Diff: LayoutTests/web-animations-api/player-cancel-finishes.html

Issue 548223002: Web-Animations: Implement idle state for AnimationPlayers (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Use known current time in finish event Created 6 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 | Source/core/animation/Animation.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/web-animations-api/player-cancel-finishes.html
diff --git a/LayoutTests/web-animations-api/player-cancel-finishes.html b/LayoutTests/web-animations-api/player-cancel-finishes.html
index 0c4d11a89e6996b9a776857dde1da074bfa6758b..e928123428f57ef4fdfe4e8a5eb9476b56bf609e 100644
--- a/LayoutTests/web-animations-api/player-cancel-finishes.html
+++ b/LayoutTests/web-animations-api/player-cancel-finishes.html
@@ -17,13 +17,13 @@ function log(message) {
results.innerHTML += message + '<br>';
}
-function validateFinishEvent(player, event) {
+function validateFinishEvent(player, event, currentTime) {
if (event.target === player) {
log('PASS: ' + player.name + ' is target');
} else {
log('FAIL: expected target named ' + player.name + ', actual target is ' + event.target);
}
- if (event.currentTime === player.currentTime) {
+ if ((currentTime !== undefined && event.currentTime == currentTime) || event.currentTime === player.currentTime) {
log('PASS: event currentTime equals player currentTime');
} else {
log('FAIL: event currentTime ' + event.currentTime + ' does not equal player currentTime ' + player.currentTime);
@@ -44,7 +44,7 @@ function onFinishTop(event) {
}
function onFinishMiddle(event) {
- validateFinishEvent(playerMiddle, event);
+ validateFinishEvent(playerMiddle, event, 0);
playerBottom.currentTime = 0;
}
« no previous file with comments | « no previous file | Source/core/animation/Animation.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698