1424895 - investigate browser_service_workers_push_service.js timeouts in non-e10s

Status: NEW

(DevTools :: General, enhancement, P3)

Description

[Ben Kelly [:bkelly, not reviewing]]

In bug 1293277 I am disabling browser_service_workers_push_service.js on non-e10s.  It seems to timeout with slight timing changes.  I think there is a race condition in the devtools service worker/push actors somewhere.

I did some initial investigation.  It seems we set the subscription correctly, but then overwrite it with undefined again when the componentDidUpdate() fires.

The test passes if I add a 100ms delay here:

https://searchfox.org/mozilla-central/rev/281d99b3d342b71d0653b872345761372d38f5c1/devtools/client/aboutdebugging/test/service-workers/push-sw.html#27

Comment 1

[Ben Kelly [:bkelly, not reviewing]]

Note, when the test passed I would get an actor that printed as:

  "[Actor pushSubscription/server1.conn0.pushSubscription54]"

But when it failed the actor always reported as:

  "[Actor pushSubscription/null]"

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

There is also a low-frequency intermittent bug for this test:

https://bugzilla.mozilla.org/show_bug.cgi?id=1307631