Bug 1267893 part 4 - Make setting the start time set 'did seek' to true draft
authorBrian Birtles <birtles@gmail.com>
Wed, 27 Apr 2016 11:34:05 +0900
changeset 356758 742ed5a19836333c5795ea010054002dc759b60d
parent 356757 a576a3a90dd544190b162e53c91c6f8a5fb756fd
child 519475 44a8dbdbb9ae842dc4ef8ead3a0b523efc296388
push id16592
push userbbirtles@mozilla.com
push dateWed, 27 Apr 2016 04:53:51 +0000
bugs1267893
milestone49.0a1
Bug 1267893 part 4 - Make setting the start time set 'did seek' to true Based on changes to the spec: https://github.com/w3c/web-animations/commit/8bf2b102dedbbc0d96b919d09a8c2d6035a14c98 MozReview-Commit-ID: GXllrLuyh8
dom/animation/Animation.cpp
testing/web-platform/meta/web-animations/timing-model/animations/set-the-animation-start-time.html.ini
--- a/dom/animation/Animation.cpp
+++ b/dom/animation/Animation.cpp
@@ -194,17 +194,17 @@ Animation::SetStartTime(const Nullable<T
 
   CancelPendingTasks();
   if (mReady) {
     // We may have already resolved mReady, but in that case calling
     // MaybeResolve is a no-op, so that's okay.
     mReady->MaybeResolve(this);
   }
 
-  UpdateTiming(SeekFlag::NoSeek, SyncNotifyFlag::Async);
+  UpdateTiming(SeekFlag::DidSeek, SyncNotifyFlag::Async);
   if (IsRelevant()) {
     nsNodeUtils::AnimationChanged(this);
   }
   PostUpdate();
 }
 
 // https://w3c.github.io/web-animations/#current-time
 Nullable<TimeDuration>
--- a/testing/web-platform/meta/web-animations/timing-model/animations/set-the-animation-start-time.html.ini
+++ b/testing/web-platform/meta/web-animations/timing-model/animations/set-the-animation-start-time.html.ini
@@ -2,12 +2,8 @@
   type: testharness
   [Setting the start time of an animation without an active timeline]
     expected: FAIL
     bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1096776
 
   [Setting an unresolved start time an animation without an active timeline does not clear the current time]
     expected: FAIL
     bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1096776
-
-  [Setting the start time updates the finished state]
-    expected: FAIL
-    bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1267893