1203652 - Update expectation data for Service Worker web platform tests that started passing after iframe onload fix.

Status: RESOLVED WORKSFORME

(Core :: DOM: Service Workers, defect, P5)

Description

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

This bug simply updates test expectation data for tests that started passing after Bug 1201632 was fixed. It is required to be able to land Bug 1201632. The patch does not fix any failures unrelated to this specific issue.

Comment 1

[Andrew Overholt [:overholt]]

From what I can tell, these are the differences from before/after the patch from bug 1201632:

/_mozilla/service-workers/service-worker/ServiceWorkerGlobalScope/unregister.https.html
---------------------------------------------------------------------------------------
PASS expected TIMEOUT Unregister controlling service worker
OK expected TIMEOUT [Parent]
/_mozilla/service-workers/service-worker/register-same-scope-different-script-url.https.html
--------------------------------------------------------------------------------------------
PASS expected TIMEOUT Register same-scope new script url effect on controller
/_mozilla/service-workers/service-worker/unregister-controller.https.html
-------------------------------------------------------------------------
PASS expected TIMEOUT Unregister prevents new controllee even if registration is still in use
OK expected TIMEOUT [Parent]
/_mozilla/service-workers/service-worker/unregister-then-register-new-script.https.html
---------------------------------------------------------------------------------------
CRASH expected TIMEOUT [Parent]
/_mozilla/service-workers/service-worker/unregister-then-register.https.html
----------------------------------------------------------------------------
FAIL expected TIMEOUT Unregister then register resolves to the original value if the registration is in use.
PASS expected TIMEOUT Unregister then register does not affect existing controllee
PASS expected TIMEOUT Unregister then register resurrects the registration

Comment 2

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Andrew, is this still valid after the web platform tests were updated some time last week?

Comment 3

[Andrew Overholt [:overholt]]

Re-running today I see:

Summary
=======

Ran 325 tests (90 parents, 235 subtests)
Expected results: 320
Unexpected results: 1 (CRASH: 1)
Skipped: 4

Unexpected Results
==================

/_mozilla/service-workers/service-worker/unregister-then-register-new-script.https.html

Comment 4

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

The crash is related to bug https://bugzilla.mozilla.org/show_bug.cgi?id=1205676
khuey is on it.

Comment 5

[Andrew Sutherland [:asuth] (he/him)]

Bot does this now; bot has updated tests.