Closed Bug 763780 Opened 8 years ago Closed 8 years ago

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

Categories

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

14 Branch
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 16

People

(Reporter: myk, Assigned: myk)

Details

(Whiteboard: [qa-])

Attachments

(1 file)

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.
Attachment #632078 - Flags: review?(benjamin)
Note that this patch will conflict trivially with the fix for bug 752666, currently on inbound.
(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.)
(In reply to :Gavin Sharp (use gavin@gavinsharp.com 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.
Priority: -- → P3
Target Milestone: --- → Firefox 16
Attachment #632078 - Flags: review?(benjamin) → review+
https://hg.mozilla.org/mozilla-central/rev/55f8b9b957c3
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
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?
Whiteboard: [qa?]
This is just an internal code change.
Whiteboard: [qa?] → [qa-]
Flags: in-moztrap-
QA Contact: jsmith
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.