NEW180257
Layout Test imported/w3c/web-platform-tests/service-workers/service-worker/claim-with-redirect.https.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=180257
Summary Layout Test imported/w3c/web-platform-tests/service-workers/service-worker/cl...
Ryan Haddad
Reported 2017-12-01 10:01:44 PST
The following layout test is flaky on macOS Debug WK2 imported/w3c/web-platform-tests/service-workers/service-worker/claim-with-redirect.https.html Flakiness Dashboard: https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=imported%2Fw3c%2Fweb-platform-tests%2Fservice-workers%2Fservice-worker%2Fclaim-with-redirect.https.html --- /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/claim-with-redirect.https-expected.txt +++ /Volumes/Data/slave/elcapitan-debug-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/claim-with-redirect.https-actual.txt @@ -1,4 +1,4 @@ -CONSOLE MESSAGE: line 27: Unhandled Promise Rejection: TypeError: undefined is not an object (evaluating 'r.update') +CONSOLE MESSAGE: Unhandled Promise Rejection: TypeError: Cannot update a null/nonexistent service worker registration FAIL Claim works after redirection to another origin assert_equals: expected (string) "updated" but got (undefined) undefined
Attachments
Ryan Haddad
Comment 1 2017-12-01 11:05:44 PST
This test recently became flaky.
Ryan Haddad
Comment 2 2017-12-01 11:06:26 PST
Alexey Proskuryakov
Comment 3 2017-12-01 11:18:41 PST
I remember that we made improvements to generating line numbers in console messages more reliably a couple years ago, but I guess there are still edge cases?
Joseph Pecoraro
Comment 4 2017-12-01 11:31:00 PST
This unhandled promise rejection gets line numbers from Exceptions. Here it looks like the exception itself is changing so apparently the new exception doesn't have line number information associated with it.
youenn fablet
Comment 5 2017-12-01 11:34:23 PST
claim is not supported yet. Hopefully, when it gets implemented, flakiness might disappear. We could use DumpConsoleLogInStdErr to make the test stable but this is probably not worth it right now.
Note You need to log in before you can comment on or make changes to this bug.