commit 5a194db083c166a92c594c011ad5c40e111df3fe
author Fredrik Söderquist <fs@opera.com> Wed Jul 04 14:27:14 2018
committer Commit Bot <commit-bot@chromium.org> Wed Jul 04 14:27:14 2018
tree 68836b3e4c88e62f1db4106bb03113826ea1cb63
parent 8d4cec73bdea16d788df75a27714567c5110f08f

Use NextFireIntervalDelta instead of NextFireInterval in SVGImage*Test

Avoids run-tripping TimeDelta -> double -> TimeDelta.

Bug: 856082
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: If4a6a0626be6341770ef01d368fc8a1ddf00de31
Reviewed-on: https://chromium-review.googlesource.com/1126115
Reviewed-by: Daniel Bratell <bratell@opera.com>
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#572560}

third_party/blink/renderer/core/svg/graphics/svg_image_test.cc

diff --git a/third_party/blink/renderer/core/svg/graphics/svg_image_test.cc b/third_party/blink/renderer/core/svg/graphics/svg_image_test.cc
index 69e92d99..805d303 100644
--- a/third_party/blink/renderer/core/svg/graphics/svg_image_test.cc
+++ b/third_party/blink/renderer/core/svg/graphics/svg_image_test.cc
@@ -115,7 +115,7 @@
   // true for shouldPauseAnimation, this will result in the timeline being
   // suspended.
   test::RunDelayedTasks(TimeDelta::FromMilliseconds(1) +
-                        TimeDelta::FromSecondsD(timer->NextFireInterval()));
+                        timer->NextFireIntervalDelta());
   EXPECT_TRUE(chrome_client.IsSuspended());
   EXPECT_FALSE(timer->IsActive());
 
@@ -149,7 +149,7 @@
   // Fire the timer/trigger a frame update. The timeline will remain
   // suspended and no frame will be scheduled.
   test::RunDelayedTasks(TimeDelta::FromMillisecondsD(1) +
-                        TimeDelta::FromSecondsD(timer->NextFireInterval()));
+                        timer->NextFireIntervalDelta());
   EXPECT_TRUE(chrome_client.IsSuspended());
   EXPECT_FALSE(timer->IsActive());
 
@@ -226,7 +226,7 @@
   // Wait for the next animation frame to be triggered, and then trigger a new
   // frame. The image animation timeline should be running.
   test::RunDelayedTasks(TimeDelta::FromMilliseconds(1) +
-                        TimeDelta::FromSecondsD(timer->NextFireInterval()));
+                        timer->NextFireIntervalDelta());
   Compositor().BeginFrame();
 
   EXPECT_FALSE(svg_image_chrome_client.IsSuspended());
@@ -236,7 +236,7 @@
   // animation timeline.)
   WebView().SetVisibilityState(mojom::PageVisibilityState::kHidden, false);
   test::RunDelayedTasks(TimeDelta::FromMilliseconds(1) +
-                        TimeDelta::FromSecondsD(timer->NextFireInterval()));
+                        timer->NextFireIntervalDelta());
 
   EXPECT_TRUE(svg_image_chrome_client.IsSuspended());
 
@@ -244,7 +244,7 @@
   // frame and resume the image animation.
   WebView().SetVisibilityState(mojom::PageVisibilityState::kVisible, false);
   test::RunDelayedTasks(TimeDelta::FromMilliseconds(1) +
-                        TimeDelta::FromSecondsD(timer->NextFireInterval()));
+                        timer->NextFireIntervalDelta());
   Compositor().BeginFrame();
 
   EXPECT_FALSE(svg_image_chrome_client.IsSuspended());