Chromium Code Reviews Chromium Code Reviews
Issue 1119683003:
Implement requestIdleCallback API (Closed)
Base URL: svn://svn.chromium.org/blink/trunk| Index: LayoutTests/virtual/threaded/fast/idle-callback/timeout.html |
| diff --git a/LayoutTests/virtual/threaded/fast/idle-callback/timeout.html b/LayoutTests/virtual/threaded/fast/idle-callback/timeout.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..2177fb365019b67761f3f3dbf7efabdf5c049611 |
| --- /dev/null |
| +++ b/LayoutTests/virtual/threaded/fast/idle-callback/timeout.html |
| @@ -0,0 +1,82 @@ |
| +<!DOCTYPE html> |
| +<html> |
| +<head> |
|
esprehn
2015/08/20 20:45:55
ditto for test simplification
rmcilroy
2015/08/21 00:03:44
Done.
|
| +<meta charset="utf-8" /> |
| +<title>window.requestIdleCallback deals with timeouts correctly</title> |
| +<link rel="author" title="Ross McIlroy" href="mailto:rmcilroy@chromium.org" /> |
| +<link rel="help" href="http://www.w3.org/TR/requestidlecallback/"/> |
| +<script src="../../../../resources/testharness.js"></script> |
| +<script src="../../../../resources/testharnessreport.js"></script> |
| +<link rel="stylesheet" href="../../resources/testharness.css" /> |
| +<script> |
| + |
| +async_test(function() { |
| + // Check whether requestIdleCallback with a timeout works when the event loop |
| + // is busy. |
| + var busy_raf_loop_iterations_remaining = 10; // Should take 20 * 40 = 400ms |
| + var idle_callback_scheduled; |
| + var idle_callback = this.step_func(function(deadline) { |
| + assert_false(deadline.didTimeout, "IdleDeadline.didTimeout MUST be false if requestIdleCallback wasn't scheduled due to a timeout"); |
| + assert_equals(busy_raf_loop_iterations_remaining, 0, "Busy rAF loop should be finished by the time we get scheduled"); |
| + this.done(); |
| + }); |
| + |
| + var busy_raf_loop_iterations_remaining = 10; // Should take 20 * 40 = 400ms |
| + requestAnimationFrame(this.step_func(function busyRAFLoop() { |
| + var start_time = performance.now(); |
| + if (!idle_callback_scheduled) { |
| + idle_callback_scheduled = start_time; |
| + requestIdleCallback(idle_callback); |
| + } |
| + |
| + // Use up the whole frame budget. |
| + while (performance.now() - start_time < 40) { } |
| + if (busy_raf_loop_iterations_remaining > 0) { |
| + busy_raf_loop_iterations_remaining--; |
| + requestAnimationFrame(busyRAFLoop); |
| + } |
| + })); |
| +}, 'requestIdleCallback not scheduled when event loop is busy.'); |
| + |
| +async_test(function() { |
| + // Check whether requestIdleCallback with a timeout works when the event loop |
| + // is busy. |
| + var busy_raf_loop_iterations_remaining = 10; // Should take 20 * 40 = 400ms |
| + var timeout = 200; |
| + var idle_callback_scheduled; |
| + var idle_callback = this.step_func(function(deadline) { |
| + var time_delta = performance.now() - idle_callback_scheduled; |
| + assert_true(time_delta >= timeout, "Should only have been run after timeout"); |
| + assert_true(time_delta - timeout <= 80, "Should have been scheduled close to timeout"); |
| + assert_true(deadline.timeRemaining() <= 0, "IdleDeadline.timeRemaining MUST be less than or equal to zero if requestIdleCallback was scheduled due to a timeout"); |
| + assert_true(deadline.didTimeout, "IdleDeadline.didTimeout MUST be true if requestIdleCallback was scheduled due to a timeout"); |
| + assert_true(busy_raf_loop_iterations_remaining > 0, "Busy rAF loop should still be going"); |
| + this.done(); |
| + }); |
| + |
| + requestAnimationFrame(this.step_func(function busyRAFLoop() { |
| + var start_time = performance.now(); |
| + if (!idle_callback_scheduled) { |
| + idle_callback_scheduled = start_time; |
| + requestIdleCallback(idle_callback, timeout); |
| + } |
| + |
| + // Use up the whole frame budget. |
| + while (performance.now() - start_time < 40) { } |
| + if (busy_raf_loop_iterations_remaining > 0) { |
| + busy_raf_loop_iterations_remaining--; |
|
esprehn
2015/08/20 20:45:55
These variables are all named wrong, but I guess y
rmcilroy
2015/08/21 00:03:44
Yes I am planning to upstream these tests so was f
|
| + requestAnimationFrame(busyRAFLoop); |
| + } |
| + })); |
| +}, 'requestIdleCallback scheduled with timeout when event loop is busy.'); |
| + |
| +</script> |
| +</head> |
| +<body> |
| +<h1>Description</h1> |
| +<p>This test validates that window.requestIdleCallback deals with timeouts correctly.</p> |
| + |
| +<div id="log"></div> |
| + |
| +</body> |
| +</html> |
