763780 - remove unnecessary application flags from webapp runtime's components.manifest

Status: RESOLVED FIXED

(Firefox Graveyard :: Webapp Runtime, defect, P3)

Description

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v1: removes application flags

The webapp runtime's components.manifest file restricts its instructions to the runtime via `application` flags.  But those are no longer necessary now that the webapp runtime files have been isolated into their own directory per bug 746156.  And the instructions registering most recent component added to the runtime (ContentPermission.js) do not have such flags.  None of them should.

Comment 1

[Myk Melez [:myk] [@mykmelez]]

Note that this patch will conflict trivially with the fix for bug 752666, currently on inbound.

Comment 2

[:Gavin Sharp [email: [e-mail]]]

(In reply to Myk Melez [:myk] [@mykmelez] from comment #0)
> the webapp runtime files have been isolated into their own directory
> per bug 746156.

Will that allow us to get rid of the equivalent lines in the Firefox/Sync manifests as well? (I never quite understood why those included b2g/mobile, since none of that code gets used in either of those.)

Comment 3

[Myk Melez [:myk] [@mykmelez]]

(In reply to :Gavin Sharp (use [e-mail] for email) from comment #2)
> (In reply to Myk Melez [:myk] [@mykmelez] from comment #0)
> > the webapp runtime files have been isolated into their own directory
> > per bug 746156.
> 
> Will that allow us to get rid of the equivalent lines in the Firefox/Sync
> manifests as well?

Sadly, no, those lines remain necessary to prevent the runtime from loading Firefox/Sync components.  To make those lines unnecessary, we would have to separate Firefox from the GRE.


> (I never quite understood why those included b2g/mobile,
> since none of that code gets used in either of those.)

Sounds like a mistake that should be fixed.

Comment 4

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 632078 [details] [diff] [review]
patch v1: removes application flags

https://hg.mozilla.org/integration/mozilla-inbound/rev/55f8b9b957c3

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/55f8b9b957c3

Comment 6

[Jason Smith [:jsmith]]

Myk - Trying to understand if it's worth verifying this bug. How is an end-user affected by this implementation? Where would they see something different? Or is this just an internal code change?

Comment 7

[Myk Melez [:myk] [@mykmelez]]

This is just an internal code change.