787468 - Webapps should set mozapps attribute on browser elements to get installed app privileges

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Web Apps (PWAs), defect, P2)

Comment 1

[Wesley Johnston (:wesj)]

Whoops. A comment is useful. B2G implemented a mozapp attribute on iframes (but really on FrameLoader elements)
https://developer.mozilla.org/en-US/docs/HTML/Element/iframe#mozapp
http://mxr.mozilla.org/mozilla-central/source/content/base/src/nsFrameLoader.cpp#1994

that's used to determine if an app should have special "installed app" priviledges. We need to do this in Fennec too.

Comment 2

[Wesley Johnston (:wesj)]

I talked to sicking about this briefly one day. I think there are questions about whether this actually gives us anything useful. I know that it enables the orientation apis, but apps can also set allowed orientations through their manifest. We set other special permissions for apps at install time:

http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/WebAppRT.js#44

as well as setting special prefs:
(written to a file here: http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/browser.js#6860
 and applied here: http://mxr.mozilla.org/mozilla-central/source/mobile/android/chrome/content/WebAppRT.js#44

B2G's implementation is very very different. cc'ing fabrice and sicking to see what permissions, api's we're missing so we can decide what we want/need to turn on/off.

Comment 3

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

I believe it'll also affect the fullscreen API.

And it'll have more effects over time. For example indexedDB will use permanent storage by default (sometime early next year probably). As will appcache though that's probably further out.

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

fixed by bug 819037. Fennec now sets the mozapps attr. Well technically we set the app ID on the docshell directly.