Closed Bug 1130570 Opened 7 years ago Closed 7 years ago

trained-to-thrill Service Worker registration throws DOMException in e10s mode


(Core :: DOM: Core & HTML, defect)

Not set



Tracking Status
e10s + ---
firefox38 --- fixed


(Reporter: bkelly, Assigned: nsm)



(3 files, 1 obsolete file)

Testing with our current best patches in this queue:

In non-e10s mode the Service Worker successfully registers.  In e10s mode, though, we get a DOMException.
Assignee: nobody → nsm.nikhil
Waiting for b2g tests to be green
This problem for e10s does not arise in treeherder because we run other worker tests so we have a runtime service around.
It does arise in e10s on individual examples as bkelly pointed out. On b2g we seem to be always running into it.
Attachment #8563015 - Flags: review?(amarchesini)
Comment on attachment 8563015 [details] [diff] [review]
Ensure we have a valid runtime service, and clear updating scopes on early return

Review of attachment 8563015 [details] [diff] [review]:

::: dom/workers/ServiceWorkerManager.cpp
@@ +2006,5 @@
>      return rv;
>    }
>    nsRefPtr<ServiceWorker> serviceWorker;
> +  RuntimeService* rs = RuntimeService::GetOrCreateService();

we already have this, right?
Attachment #8563015 - Flags: review?(amarchesini) → review+
Attached patch iframe vs (obsolete) — Splinter Review
Another reason why we fail in b2g desktop/emulator is that we don't have the support for when the code runs into an app. Here we use an iframe.
Attachment #8563306 - Flags: review?(nsm.nikhil)
Comment on attachment 8563306 [details] [diff] [review]
iframe vs

Review of attachment 8563306 [details] [diff] [review]:

::: dom/workers/test/serviceworkers/mochitest.ini
@@ +1,2 @@
> +skip-if = android_version == "10" # bug 1056702

FYI, I removed this in my patches since we use 11 on treeherder now. They pass on 11, not sure if we still need this.
Attachment #8563306 - Flags: review?(nsm.nikhil) → review+
Do you want to land my patch with yours?
Attachment #8563306 - Attachment is obsolete: true
Flags: needinfo?(nsm.nikhil)
Yes, about to push to try.
Flags: needinfo?(nsm.nikhil)
also needs Bug 1130065 before this can land.
Ugh, I'm not sure why we continue to have b2g failures. baku, can you give this a look since you are currently surrounded by b2g experts? Thanks!
Flags: needinfo?(amarchesini)
sorry i had to to back this out in - seems one of this changes was causing on multiple desktop platforms test failures like

that became frequent to perma failures starting with this csets. Could you take a look at this, thanks!
Closed: 7 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Flags: needinfo?(amarchesini)
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.