Commit c57e36e2 authored by Yi Gu's avatar Yi Gu Committed by Commit Bot

[Code Health] Fix incorrect test name

wpt/web-animations/timing-model/animations/set-the-animation-start-time.html
was renamed in [1] but the name change was not reflected elsewhere.

In addition, the expectation file for the test on Android seems
unnecessary.

[1] https://chromium.googlesource.com/chromium/src/+/bf6c3b061813db88ca12c59690cece60d0b672f7

Change-Id: I6a640c3d56203380ad19ca3b1bb1278cc6971bd8
Reviewed-on: https://chromium-review.googlesource.com/c/1477938Reviewed-by: default avatarDirk Pranke <dpranke@chromium.org>
Commit-Queue: Yi Gu <yigu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#634404}
parent 21dba6f0
...@@ -316,7 +316,7 @@ external/wpt/web-animations/interfaces/KeyframeEffect/composite.html ...@@ -316,7 +316,7 @@ external/wpt/web-animations/interfaces/KeyframeEffect/composite.html
external/wpt/web-animations/interfaces/KeyframeEffect/spacing.html external/wpt/web-animations/interfaces/KeyframeEffect/spacing.html
external/wpt/web-animations/timing-model/animation-effects/active-time.html external/wpt/web-animations/timing-model/animation-effects/active-time.html
external/wpt/web-animations/timing-model/animations/finishing-an-animation.html external/wpt/web-animations/timing-model/animations/finishing-an-animation.html
external/wpt/web-animations/timing-model/animations/set-the-animation-start-time.html external/wpt/web-animations/timing-model/animations/setting-the-start-time-of-an-animation.html
external/wpt/web-animations/timing-model/time-transformations/transformed-progress.html external/wpt/web-animations/timing-model/time-transformations/transformed-progress.html
external/wpt/webaudio/the-audio-api/the-audiobuffersourcenode-interface/audiobuffersource-ended.html external/wpt/webaudio/the-audio-api/the-audiobuffersourcenode-interface/audiobuffersource-ended.html
external/wpt/webaudio/the-audio-api/the-audiobuffersourcenode-interface/audiobuffersource-playbackrate-zero.html external/wpt/webaudio/the-audio-api/the-audiobuffersourcenode-interface/audiobuffersource-playbackrate-zero.html
......
...@@ -19,7 +19,7 @@ Guidelines for writing tests ...@@ -19,7 +19,7 @@ Guidelines for writing tests
Instead, try something like: Instead, try something like:
> * `/web-animations/timing-model/animations/set-the-animation-start-time.html`<br> > * `/web-animations/timing-model/animations/setting-the-start-time-of-an-animation.html`<br>
> Tests all the branches and inputs to the procedure as defined in the > Tests all the branches and inputs to the procedure as defined in the
> spec (using the `Animation.startTime` API). > spec (using the `Animation.startTime` API).
> * `/web-animations/interfaces/Animation/startTime.html`<br> > * `/web-animations/interfaces/Animation/startTime.html`<br>
......
This is a testharness.js-based test.
FAIL Setting the start time of an animation without an active timeline assert_equals: Setting the start time clears the current time expected (object) null but got (number) -463.4576666666703
FAIL Setting an unresolved start time an animation without an active timeline does not clear the current time assert_equals: Start time remains unresolved expected (object) null but got (number) 0
PASS Setting the start time clears the hold time
FAIL Setting an unresolved start time sets the hold time assert_approx_equals: Hold time is set after start time is made unresolved expected 1000 +/- 0.0005 but got 536.5423333333297
FAIL Setting the start time resolves a pending ready promise assert_true: Animation is in play-pending state expected true got undefined
FAIL Setting the start time resolves a pending pause task assert_true: Animation is in pause-pending state expected true got undefined
FAIL Setting the start time updates the finished state assert_greater_than: Setting the start time updated the finished state with the 'did seek' flag set to true expected a number greater than 100000 but got 100000
Harness: the test ran to completion.
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment