130877 - Add XPCOM support for per application component registry

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Chak Nanga]

This feature is needed to support MRE based embedding applications. The changes
involve the following:

1. Modify ComponentManager to look for XPCOM components in both the application
specific components directory and in the MRE specific component directory.
The component.reg file must be created in the mre *application*'s directory

2. Modify xptiInterfaceInfoManager to look for .xpt files in both the
application specific components directory and in the MRE specific component
directory. The xpti.dat file must be created in the mre *application*'s
"components" directory

Comment 1

[Chak Nanga]

Attachment: Patch to create app specific component registry

This patch addresses the two issues mentioned above in the intial bug report.

John : Doug's taken a look at this earlier. Could you please take a look at the
xptiInterfaceInfoManager changes and comment?

Thanks
Chak

Comment 2

[Chak Nanga]

Hi John: 

Could you please review the patch above when you get a chance...

Thanks
Chak

Comment 3

[Doug Turner (:dougt)]

pushing milestone out based on requirement feedback from Chak and Jud.

Comment 4

[Asaf Amit]

We tried using the patch. However, we could not get into a situation where
the xpti.dat in the MRE application components dir will contain the infromation
about the xpt files in Mozilla components dir. 

The second requirement in the bug description is "The xpti.dat file must be
created in the mre *application*'s "components" directory". This is not happening.

Comment 5

[Chak Nanga]

Attachment: Updated patch to work with the latest trunk...

Doug : We need to discuss the ordering issue as hilited below in the attached
patch:

+    
+    // Ordering is VERY important here and should be discussed.
+    // xxx

Comment 6

[Doug Turner (:dougt)]

Comment on attachment 85820 [details] [diff] [review]
Updated patch to work with the latest trunk...

remove that comment.  we discussed it.	with that, r=

Comment 7

[Chak Nanga]

Attachment: Updated patch to address dougt's comments above

Added some comments as well...

Comment 8

[Chak Nanga]

Comment on attachment 86502 [details] [diff] [review]
Updated patch to address dougt's comments above

Carrying over dougt's r=

Comment 9

[Mike Shaver (:shaver emeritus)]

Comment on attachment 86502 [details] [diff] [review]
Updated patch to address dougt's comments above

Please use sMRE_Directory and such, like you do with sOS_*.  With that change,
sr=shaver.

Comment 10

[Chak Nanga]

Landed patch on the trunk

Comment 11

[Chak Nanga]

*** Bug 130880 has been marked as a duplicate of this bug. ***

Comment 12

[Marek Z. Jeziorek]

batch: adding topembed per Gecko2 document
http://rocknroll.mcom.com/users/marek/publish/Gecko/Gecko2Tasks.html