762007 - mozIDOMApplicationMgmt inherits builtinclass so it must be implemented in C++, or changed to not inherit builtinclass

Status: RESOLVED FIXED

Description

[Olli Pettay [:smaug] (PTO-ish Feb 16-23)]

Comment 1

[Olli Pettay [:smaug] (PTO-ish Feb 16-23)]

for now, could we just change  mozIDOMApplicationMgmt to not inherit nsIDOMEventTarget?

Comment 2

[Olli Pettay [:smaug] (PTO-ish Feb 16-23)]

...since the implementation doesn't seem to implement nsIDOMEventTarget anyway.

Comment 3

[Olli Pettay [:smaug] (PTO-ish Feb 16-23)]

Created attachment 630540 [details] [diff] [review]
WIP

Comment 4

[Olli Pettay [:smaug] (PTO-ish Feb 16-23)]

Why are webapp implementation bugs DOM bugs?

Comment 5

[Jason Smith [:jsmith]]

(In reply to Olli Pettay [:smaug] from comment #4)
> Why are webapp implementation bugs DOM bugs?

Core --> DOM:Mozilla Extensions is where bugs go for issues related to underlying mozapps API. Checking the changelog for the file being worked on, this file seems to be attached to that.

Firefox --> Web Apps is front-end work for the end user for the install lifecycle for apps natively (e.g. install app, launch app).

Myk's description a little while back summarizes it as:

"The webapp runtime for Firefox, which comprises: the stub executable launcher (i.e. the binary and related files that Firefox creates for each webapp you install to your local computer) and the XUL shell (i.e. the XULRunner application that the launcher uses to load a webapp and provide it with native integration APIs). Note: bugs about Gecko's navigator.mozApps DOM API (i.e. the API that web pages use to request webapp installation) go in Core / DOM: Mozilla Extensions; bugs about Firefox's webapp installer (i.e. the feature that creates a launcher when the user approves an installation request) go in Firefox / Web Apps. "

Comment 6

[Jason Smith [:jsmith]]

(In reply to Jason Smith [:jsmith] from comment #5)
> "The webapp runtime for Firefox, which comprises: the stub executable
> launcher (i.e. the binary and related files that Firefox creates for each
> webapp you install to your local computer) and the XUL shell (i.e. the
> XULRunner application that the launcher uses to load a webapp and provide it
> with native integration APIs). Note: bugs about Gecko's navigator.mozApps
> DOM API (i.e. the API that web pages use to request webapp installation) go
> in Core / DOM: Mozilla Extensions; bugs about Firefox's webapp installer
> (i.e. the feature that creates a launcher when the user approves an
> installation request) go in Firefox / Web Apps. "

Note - this is pulled from Firefox --> Webapp Runtime

Comment 7

[Olli Pettay [:smaug] (PTO-ish Feb 16-23)]

Created attachment 630573 [details] [diff] [review]
WIP2

Need to fix the test, and also the event interface.

Comment 8

[Olli Pettay [:smaug] (PTO-ish Feb 16-23)]

Created attachment 630593 [details] [diff] [review]
patch

Comment 9

[Olli Pettay [:smaug] (PTO-ish Feb 16-23)]

https://hg.mozilla.org/mozilla-central/rev/794751c0c8f4