Remove nsIChannel CorsMode flag and set Request.mode based on nsILoadInfo securityFlags instead




4 years ago
a year ago


(Reporter: bkelly, Unassigned)


(Blocks 1 bug)

Firefox Tracking Flags

(Not tracked)


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.


4 years ago
Blocks: 1182120
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.
Blocks: ServiceWorkers-postv1
No longer blocks: ServiceWorkers-v1


4 years ago
Blocks: 1182113


4 years ago
No longer blocks: 1182113
This is a compat issue because the legacy code paths don't reliably provide the correct FetchEvent.request values.
Severity: normal → minor
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.