From 0fa54b80d488571f10d8a94f6740006f0bf4957c Mon Sep 17 00:00:00 2001 From: tsepez Date: Thu, 4 Aug 2016 12:07:28 -0700 Subject: Beef up timer cancellation tests Adds more questionable invocations of ClearTimeOut(). Also, checking that nothing happened is fragile. Log at least one thing to show that the code ran. Review-Url: https://codereview.chromium.org/2218473002 --- testing/resources/bug_620428.in | 14 ++++++++++++++ 1 file changed, 14 insertions(+) (limited to 'testing/resources/bug_620428.in') diff --git a/testing/resources/bug_620428.in b/testing/resources/bug_620428.in index 1942305173..050488b1ee 100644 --- a/testing/resources/bug_620428.in +++ b/testing/resources/bug_620428.in @@ -73,8 +73,22 @@ function fireInterval() { } var timer = app.setTimeOut("fireTimeOut()", 3000); var interval = app.setInterval("fireInterval()", 1000); +// Clear timers before they fire. app.clearTimeOut(timer); app.clearInterval(interval); +// Check that clearing a cleared timer is ok. +app.clearTimeOut(timer); +app.clearInterval(interval); +// Check that invoking the wrong clear method is ok +var timer = app.setTimeOut("fireTimeOut()", 3000); +var interval = app.setInterval("fireInterval()", 1000); +app.clearTimeOut(interval); +app.clearInterval(timer); +// Check that clearing a non-timer is ok. +app.clearTimeOut({"size": 42}); +app.clearInterval({"size": 42}); +// Be sure all of this code ran to completion. +app.alert("done"); endstream endobj {{xref}} -- cgit v1.2.3