Once bug 1182535 updates enough of the network callsites we should switch the ServiceWorkerManager code to using nsILoadInfo::GetSecurityFlags() to set the Request.mode value. Once this is done we can remove the nsIChannel CorsMode flag. The new nsILoadInfo securityFlags is more robust and being integrated with the rest of the tree. For example, the securityFlags will actually reflect same-origin policies which are currently not reflected in actual values on CorsMode. Note, we will still need to exempt the mode for document nsIContentPolicy values since navigations need to be same-origin, but the security flags won't reflect that.
I have a patch in bug 1184967 that adds the conversion from securityFlags.
Ben said elsewhere that this is nice but not entirely necessary for the initial shipping of SW. He noted some changes that he's made and others still needed but I didn't catch them quickly enough to type them here.
This is a compat issue because the legacy code paths don't reliably provide the correct FetchEvent.request values.
You need to log in before you can comment on or make changes to this bug.