Closed Bug 1233911 Opened 8 years ago Closed 8 years ago

NativeApps.js needs to use user context origin attributes correctly

Categories

(Firefox Graveyard :: Web Apps, defect)

Product:
Firefox Graveyard
Component:
Web Apps
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: huseby, Unassigned)

References

Details

(Whiteboard: [userContextId][OA]) 

In the file toolkit/webapps/NativeApp.jsm there are calls to createCodebasePrincipal that need to be fixed to use the correct user context origin attributes.  Here's the offending code:

> toolkit/webapps/NativeApp.jsm
>   line 459
>     let principal =
>       aIconURI.schemeIs("chrome") ?
>         Services.scriptSecurityManager.getSystemPrincipal() :
>         Services.scriptSecurityManager.createCodebasePrincipal(aIconURI, {});
Jonas, what's the right way to go in this file?  Should NativeApp.jsm use user context isolation?
Flags: needinfo?(jonas)
Component: Web Apps → DOM
Product: Firefox → Core
Component: DOM → Web Apps
Product: Core → Firefox
Whiteboard: [userContextId]
Flags: needinfo?(jonas)
Per bug 1238079, we're going to disable the desktop web runtime and remove it
from the codebase, so we won't fix these bugs in the integration between Firefox and the runtime.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Product: Firefox → Firefox Graveyard
(In reply to Myk Melez [:myk] [@mykmelez] from comment #2)
> Per bug 1238079, we're going to disable the desktop web runtime and remove it
> from the codebase, so we won't fix these bugs in the integration between
> Firefox and the runtime.

Do you have a rough timeline for this?  FF 47/48?  Or later?  Thanks!
Whiteboard: [userContextId] → [userContextId][OA]
You need to log in before you can comment on or make changes to this bug.