1291291 - Remove dom/apps

Status: RESOLVED DUPLICATE of bug 1318218

(Core Graveyard :: DOM: Apps, defect)

Description

[[:fabrice] Fabrice Desré]

The app runtime won't be used anymore, so it's fine to remove it and all the dependencies.

Comment 1

[[:fabrice] Fabrice Desré]

jst, do you have someone that can tackle that? That will lead to a lot of code removal all over the place, since we won't need the appId and appStatus on the principals.

Comment 2

[Gabriele Svelto [:gsvelto] (PTO)]

I've got some spare cycles, I'll take care of this.

Comment 3

[Gabriele Svelto [:gsvelto] (PTO)]

Attachment: [PATCH WIP] Remove dom/apps

Work in progress patch... this is *big*, and it's probably like ~20% of the work that needs to be done. I'll try to split it up otherwise keeping it updated and reviewing it will be a nightmare.

Note, with these changes not even desktop Firefox build correctly. This is what's known as "machete programming": you're in the jungle, you cut a path through it removing everything that's in the way and only when you're on the other side you fix up the rest (hopefully).

Comment 4

[(no longer active)]

I propose we do things piecemeal in order to avoid a boil the ocean approach.

Over in bug 1261019, I'm working on removing navigator.mozApps.  That removes a good chunk of code inside dom/apps that either implements mozApps and the APIs attached to it, and the code that purely used to support that implementation.  Currently, that patch leaves the following files/dirs in dom/apps:

AppsService.js           AppsServiceChild.jsm     MessageBroadcaster.jsm   PermissionsInstaller.jsm UserCustomizations.jsm   tests
AppsService.manifest     AppsUtils.jsm            OperatorApps.jsm         PermissionsTable.jsm     moz.build

Some of this code implements the apps service which is used in various parts of the tree.  Eliminating those usages is super helpful (bug 1255046.)

Some of the other JSMs left are stufff that are used one off in some random part of the code which we can remove as well in small patches.

What do you think, Gabriele?  If you agree with this plan, bug 1255046 is a self contained work item while I'm nuking out mozApps.  :-)

Comment 5

[Gabriele Svelto [:gsvelto] (PTO)]

(In reply to :Ehsan Akhgari from comment #4)
> Over in bug 1261019, I'm working on removing navigator.mozApps.  That
> removes a good chunk of code inside dom/apps that either implements mozApps
> and the APIs attached to it, and the code that purely used to support that
> implementation.  Currently, that patch leaves the following files/dirs in
> dom/apps:
> 
> AppsService.js           AppsServiceChild.jsm     MessageBroadcaster.jsm  
> PermissionsInstaller.jsm UserCustomizations.jsm   tests
> AppsService.manifest     AppsUtils.jsm            OperatorApps.jsm        
> PermissionsTable.jsm     moz.build
> 
> Some of this code implements the apps service which is used in various parts
> of the tree.  Eliminating those usages is super helpful (bug 1255046.)

I see. I've started cutting deeply here because that's an easy way to spot breakage .and thus be able to fix it. I intend to split this into multiple patches though otherwise it'll be a nightmare.

> Some of the other JSMs left are stufff that are used one off in some random
> part of the code which we can remove as well in small patches.
> 
> What do you think, Gabriele?  If you agree with this plan, bug 1255046 is a
> self contained work item while I'm nuking out mozApps.  :-)

Sounds good to me! Don't be afraid of bit-rotting the stuff I'm doing here since it'll need rebase anyway.

Comment 6

[(no longer active)]

Sounds good!  FWIW I'm pretty close to a patch which is landable to remove navigator.mozApps, but this is lagging a bit since I'm fixing up the remaining issues in my free time.  The current state of things on try is at <https://treeherder.mozilla.org/#/jobs?repo=try&revision=201b0ebf20b2e73392a335bbde15767a4f12c1e2>.

Comment 7

[Gabriele Svelto [:gsvelto] (PTO)]

Right now I don't have time to work on this due to other deadlines; I'll resume at a later point if nobody picks it up in the meantime.

Comment 8

[(no longer active)]

I've finished removing this directory on inbound in bug 1318218.