207060 2020-01-31 11:20:00 -0800 [ Mojave wk2 Release ] imported/w3c/web-platform-tests/html/webappapis/timers/type-long-setinterval.html flaky failure 2020-05-07 15:26:29 -0700 1 1 1 Unclassified WebKit WebCore Misc. WebKit Nightly Build Unspecified Unspecified RESOLVED DUPLICATE 172052 InRadar P2 Normal --- 1 Lawrence.j webkit-unassigned ap darin tsavell webkit-bot-watchers-bugzilla webkit-bug-importer oldest_to_newest 1613497 0 Lawrence.j 2020-01-31 11:20:00 -0800 imported/w3c/web-platform-tests/html/webappapis/timers/type-long-setinterval.html Description: This test is lightly flaky failing on Mojave wk2 release. History: https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fhtml%2Fwebappapis%2Ftimers%2Ftype-long-setinterval.html&limit=50000&platform=mac Diff: --- /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/html/webappapis/timers/type-long-setinterval-expected.txt +++ /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/html/webappapis/timers/type-long-setinterval-actual.txt @@ -1,3 +1,4 @@ +CONSOLE MESSAGE: line 3463: Error: assert_unreached: Reached unreachable code PASS Type long timeout for setInterval 1613498 1 webkit-bug-importer 2020-01-31 11:20:24 -0800 <rdar://problem/59068308> 1613507 2 389386 Lawrence.j 2020-01-31 11:37:33 -0800 Created attachment 389386 Update Test Expectations 1613512 3 389386 tsavell 2020-01-31 11:41:28 -0800 Comment on attachment 389386 Update Test Expectations Clearing flags on attachment: 389386 Committed r255514: <https://trac.webkit.org/changeset/255514> 1613768 4 ap 2020-02-01 17:57:03 -0800 > interval = setInterval(next, Math.pow(2, 32)); > setTimeout(assert_unreached, 100); I think that this tests a zero delay interval vs. a 100 ms delay timeout. Are these supposed to be on the same timeline? 1613769 5 ap 2020-02-01 18:11:45 -0800 I guess clamping makes it test something different from what the author intended, but flakiness still seems surprising to me. 1613770 6 darin 2020-02-01 18:16:44 -0800 (In reply to Alexey Proskuryakov from comment #4) > > interval = setInterval(next, Math.pow(2, 32)); > > setTimeout(assert_unreached, 100); > > I think that this tests a zero delay interval vs. a 100 ms delay timeout. > Are these supposed to be on the same timeline? Yes, setInterval and setTimeout should be on the same timeline. 1650373 7 ryanhaddad 2020-05-07 15:26:29 -0700 *** This bug has been marked as a duplicate of bug 172052 *** 389386 2020-01-31 11:37:33 -0800 2020-01-31 11:41:28 -0800 Update Test Expectations bug-207060-20200131113732.patch text/plain 1624 Lawrence.j U3VidmVyc2lvbiBSZXZpc2lvbjogMjU1NTEyCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggNWQ0OTVjMDU5MDliZTI5ZjM1OWVm YmM4MTg4NTZlYjJhYTM2MjFjMy4uYjFlNTBhMWQ3ODZmMDJkYTQwMTlmMWVmMjc3YTQ1NjljZGY1 N2RiMiAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDEyIEBACisyMDIwLTAxLTMxICBKYXNvbiBMYXdyZW5jZSAg PGxhd3JlbmNlLmpAYXBwbGUuY29tPgorCisgICAgICAgIFsgTW9qYXZlIHdrMiBSZWxlYXNlIF0g aW1wb3J0ZWQvdzNjL3dlYi1wbGF0Zm9ybS10ZXN0cy9odG1sL3dlYmFwcGFwaXMvdGltZXJzL3R5 cGUtbG9uZy1zZXRpbnRlcnZhbC5odG1sIGZsYWt5IGZhaWx1cmUKKyAgICAgICAgaHR0cHM6Ly9i dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIwNzA2MAorCisgICAgICAgIFVucmV2aWV3 ZWQgdGVzdCBnYXJkZW5pbmcuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9tYWMtd2syL1Rlc3RFeHBl Y3RhdGlvbnM6CisKIDIwMjAtMDEtMzEgIEphY29iIFVwaG9mZiAgPGphY29iX3VwaG9mZkBhcHBs ZS5jb20+CiAKICAgICAgICAgWyBtYWNPUyB3azEgXSBpbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3Jt LXRlc3RzL0luZGV4ZWREQi9pbnRlcmxlYXZlZC1jdXJzb3JzLXNtYWxsLmh0bWwgaXMgZmxha3kg ZmFpbGluZwpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjLXdrMi9UZXN0RXhw ZWN0YXRpb25zIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjLXdrMi9UZXN0RXhwZWN0YXRpb25z CmluZGV4IDc3ZWQ5NDVhNzVjMzQwNTc1ODRmYmQzNzNhOTI4MzBmNjJhNzRmZWUuLmY1OGY3YTE1 NmUyOTM5ZGUyOGY5M2QzOGVjNDc3M2RkZTg1NWRlZDIgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3Rz L3BsYXRmb3JtL21hYy13azIvVGVzdEV4cGVjdGF0aW9ucworKysgYi9MYXlvdXRUZXN0cy9wbGF0 Zm9ybS9tYWMtd2syL1Rlc3RFeHBlY3RhdGlvbnMKQEAgLTk1MCw0ICs5NTAsNiBAQCB3ZWJraXQu b3JnL2IvMjA2ODM5IGh0dHAvdGVzdHMvZnJhbWUtdGhyb3R0bGluZy9yYWYtdGhyb3R0bGUtaW4t Y3Jvc3Mtb3JpZ2luLXN1YgogCiB3ZWJraXQub3JnL2IvMjA3MDAxIHdlYnJ0Yy92aWRlby1hdXRv cGxheS5odG1sIFsgUGFzcyBGYWlsdXJlIF0KIAotd2Via2l0Lm9yZy9iLzIwNzAwMyB0aWxlZC1k cmF3aW5nL3Njcm9sbGluZy9zY3JvbGwtc25hcC9zY3JvbGwtc25hcC1tYW5kYXRvcnktb3ZlcmZs b3cuaHRtbCBbIFBhc3MgRmFpbHVyZSBdClwgTm8gbmV3bGluZSBhdCBlbmQgb2YgZmlsZQord2Vi a2l0Lm9yZy9iLzIwNzAwMyB0aWxlZC1kcmF3aW5nL3Njcm9sbGluZy9zY3JvbGwtc25hcC9zY3Jv bGwtc25hcC1tYW5kYXRvcnktb3ZlcmZsb3cuaHRtbCBbIFBhc3MgRmFpbHVyZSBdCisKK3dlYmtp dC5vcmcvYi8yMDcwNjAgaW1wb3J0ZWQvdzNjL3dlYi1wbGF0Zm9ybS10ZXN0cy9odG1sL3dlYmFw cGFwaXMvdGltZXJzL3R5cGUtbG9uZy1zZXRpbnRlcnZhbC5odG1sIFsgUGFzcyBGYWlsdXJlIF0K XCBObyBuZXdsaW5lIGF0IGVuZCBvZiBmaWxlCg==