Commit 66481283 authored by Stephen McGruer's avatar Stephen McGruer Committed by Commit Bot

Speculative fix to de-flake AnimationKeyframeEffectV8Test

It appears like calling ResetTimeForTesting is not actually needed for
this test to pass. Since it is causing flake due to the double/TimeTick
round-trips (see bug), just remove it.

Bug: 838676
Change-Id: Ice3e2c88e3afbdf07c52abfb8bffc9d8e0d0c8b1
Reviewed-on: https://chromium-review.googlesource.com/1042380Reviewed-by: default avatarRobert Flack <flackr@chromium.org>
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#555902}
parent c5c26c1c
......@@ -31,12 +31,7 @@ class KeyframeEffectTest : public PageTestBase {
void SetUp() override {
PageTestBase::SetUp(IntSize());
element = GetDocument().CreateElementForBinding("foo");
GetDocument().GetAnimationClock().ResetTimeForTesting(
base::TimeTicks() +
base::TimeDelta::FromSecondsD(GetDocument().Timeline().ZeroTime()));
GetDocument().documentElement()->AppendChild(element.Get());
EXPECT_EQ(0, GetDocument().Timeline().currentTime());
}
KeyframeEffectModelBase* CreateEmptyEffectModel() {
......
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