Bug 201661 - Layout Test imported/w3c/web-platform-tests/service-workers/service-worker/getregistrations.https.html is flaky
Summary: Layout Test imported/w3c/web-platform-tests/service-workers/service-worker/ge...
Status: RESOLVED DUPLICATE of bug 201843
Alias: None
Product: WebKit
Classification: Unclassified
Component: Service Workers (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL: https://github.com/web-platform-tests...
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-09-10 15:21 PDT by Ryan Haddad
Modified: 2019-09-17 14:44 PDT (History)
8 users (show)

See Also:


Attachments
Patch (3.92 KB, patch)
2019-09-13 11:01 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan Haddad 2019-09-10 15:21:19 PDT
The following layout test is flaky on Mojave Release WK2

imported/w3c/web-platform-tests/service-workers/service-worker/getregistrations.https.html

Probable cause:

It seems like this started after a service worker WPT update in https://trac.webkit.org/changeset/249672/webkit

Flakiness Dashboard:

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=imported%2Fw3c%2Fweb-platform-tests%2Fservice-workers%2Fservice-worker%2Fgetregistrations.https.html

--- /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/getregistrations.https-expected.txt
+++ /Volumes/Data/slave/mojave-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/service-workers/service-worker/getregistrations.https-actual.txt
@@ -1,4 +1,3 @@
-
 
 PASS registrations are not returned following unregister 
 PASS Register then getRegistrations
Comment 1 Radar WebKit Bug Importer 2019-09-10 16:01:52 PDT
<rdar://problem/55240851>
Comment 2 Chris Dumez 2019-09-13 11:01:00 PDT
Created attachment 378737 [details]
Patch
Comment 3 Alex Christensen 2019-09-13 11:34:16 PDT
Comment on attachment 378737 [details]
Patch

Are we going to upstream this?
Comment 4 Chris Dumez 2019-09-13 11:40:29 PDT
(In reply to Alex Christensen from comment #3)
> Comment on attachment 378737 [details]
> Patch
> 
> Are we going to upstream this?

Well, I am going to try. Maybe they can help find the root cause upstream.
Comment 5 Chris Dumez 2019-09-13 11:44:59 PDT
(In reply to Chris Dumez from comment #4)
> (In reply to Alex Christensen from comment #3)
> > Comment on attachment 378737 [details]
> > Patch
> > 
> > Are we going to upstream this?
> 
> Well, I am going to try. Maybe they can help find the root cause upstream.

https://github.com/web-platform-tests/wpt/pull/19047
Comment 6 WebKit Commit Bot 2019-09-13 12:17:28 PDT
Comment on attachment 378737 [details]
Patch

Clearing flags on attachment: 378737

Committed r249850: <https://trac.webkit.org/changeset/249850>
Comment 7 WebKit Commit Bot 2019-09-13 12:17:29 PDT
All reviewed patches have been landed.  Closing bug.
Comment 8 Chris Dumez 2019-09-17 14:43:21 PDT
Committed r249983: <https://trac.webkit.org/changeset/249983>
Comment 9 Chris Dumez 2019-09-17 14:43:35 PDT
Reverted in <https://trac.webkit.org/changeset/249983>.
Comment 10 Chris Dumez 2019-09-17 14:44:05 PDT
Seems to be fixed by the testharness.js updated via Bug 201843.

*** This bug has been marked as a duplicate of bug 201843 ***