Closed Bug 1255692 Opened 9 years ago Closed 9 years ago

Add another test for calling ServiceWorkerRegistration.showNotification

Categories

(Core :: DOM: Service Workers, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(1 file)

No description provided.
wchen already reviewed this
backed this out for perma test failures like https://treeherder.mozilla.org/logviewer.html#?job_id=23532088&repo=mozilla-inbound starting with this push
Flags: needinfo?(bzbarsky)
I have no idea why this particular change caused that, other than the fact that the u2f tests are broken by design... I filed bug 1255784 on them. But why would this test have affected them at all? The u2f tests run in M12 on that platform ("Android 2.3 API9 opt") and serviceworker tests, including the one I added run in M13... The answer is that without my patch M12 runs dom/u2f/tests/test_util_methods.html and M13 runs, in order, dom/u2f/tests/test_frame.html and dom/u2f/tests/test_no_token.html. With my patch, on the other hand, M12 runs, in order, dom/u2f/tests/test_no_token.html and dom/u2f/tests/test_util_methods.html while M13 runs dom/u2f/tests/test_frame.html. So it sounds like test_no_token depends on test_frame running before it or something.
Depends on: 1255784
Flags: needinfo?(bzbarsky)
Flags: needinfo?(bzbarsky)
Flags: needinfo?(bzbarsky)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: