If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

NS_APPSTARTUP_CONTRACTID should exist in nsToolkitCompsCID.h and not nsXPFEComponentsCID.h (kill nsXPFEComponentsCID.h)

RESOLVED FIXED in mozilla1.9.3a3

Status

()

Toolkit
Startup and Profile System
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

unspecified
mozilla1.9.3a3
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
Created attachment 430256 [details] [diff] [review]
The fix

NS_APPSTARTUP_CONTRACTID is now used exclusively in toolkit despite the fact it is still defined in nsXPFEComponentsCID.h:

http://mxr.mozilla.org/comm-central/search?string=NS_APPSTARTUP_CONTRACTID

NS_COMMANDLINESERVICE_CONTRACTID is also defined in nsXPFEComponentsCID.h but that isn't used anywhere:

http://mxr.mozilla.org/comm-central/search?string=+NS_COMMANDLINESERVICE_CONTRACTID&find=&findi=&filter=^[^\0]*%24&hitlimit=&tree=comm-central

Therefore, we should move NS_APPSTARTUP_CONTRACTID to nsToolkitCompsCID.h and then we can just kill nsXPFEComponentsCID.h.

Attaching patch to do just this. Tested on try server and it built just fine.
(Assignee)

Updated

8 years ago
Attachment #430256 - Attachment is patch: true
Attachment #430256 - Attachment mime type: application/octet-stream → text/plain
Attachment #430256 - Flags: review?(benjamin)
(Assignee)

Updated

8 years ago
Blocks: 550146

Updated

8 years ago
Attachment #430256 - Flags: review?(benjamin) → review+
(Assignee)

Comment 1

8 years ago
Checked in: http://hg.mozilla.org/mozilla-central/rev/c2060d652ccb
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a3
You need to log in before you can comment on or make changes to this bug.