Add MRE component registration support to NS_InitEmbedding()

VERIFIED FIXED in mozilla1.1alpha

Status

()

Core
Embedding: APIs
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: Chak Nanga, Assigned: Chak Nanga)

Tracking

Trunk
mozilla1.1alpha
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Assignee)

Description

16 years ago
If an embedding application indicates that it's using an MRE(via it's
FileLocationProvider) we need to query that location and AutoRegister components
in that directory as well.
(Assignee)

Comment 1

16 years ago
Created attachment 74338 [details] [diff] [review]
Patch to AutoRegister components in the MRE dir

I've made the current changes to NS_InitEmbedding().

We probably need to move this into an NS_InitEmbedding2() since
NS_InitEmbedding() is FROZEN?
(Assignee)

Updated

16 years ago
Blocks: 129573

Comment 2

16 years ago
yea, we can't whack frozen ifaces.

Updated

16 years ago
QA Contact: mdunn → depstein
(Assignee)

Comment 3

16 years ago
Created attachment 74753 [details] [diff] [review]
Updated patch - added NS_InitEmbedding2()

Adding an additional API named "NS_InitEmbedding2()" which encapsulates the
registration of MRE components.
(Assignee)

Comment 4

16 years ago
Comment on attachment 74338 [details] [diff] [review]
Patch to AutoRegister components in the MRE dir

This patch is obsolete now...
Attachment #74338 - Attachment is obsolete: true

Comment 5

16 years ago
pushing milestone out based on requirement feedback from Chak and Jud.
Target Milestone: --- → mozilla1.1alpha
(Assignee)

Comment 6

16 years ago
Created attachment 85803 [details] [diff] [review]
Updated patch to work with the latest trunk

1. Obsoleting old patch
2. Updated patch which works with the trunk as of today
Attachment #74753 - Attachment is obsolete: true

Comment 7

16 years ago
chak, rpotts, and i spoke about this a bit and we like patch (id=74338) best. 
There is no api change and the semantics remain the same for existing consumers.  
(Assignee)

Comment 8

16 years ago
Created attachment 86496 [details] [diff] [review]
Getting rid of NS_InitEmbedding2() per doug's comments above

Reverting back to the very fist patch - with some added comments now.
Attachment #85803 - Attachment is obsolete: true
(Assignee)

Comment 9

16 years ago
Cc:ing rpotts
(Assignee)

Comment 10

16 years ago
Comment on attachment 86496 [details] [diff] [review]
Getting rid of NS_InitEmbedding2() per doug's comments above

DougTnscp (1:54:14 PM): r=dougt via aim
Attachment #86496 - Flags: review+

Comment 11

16 years ago
Comment on attachment 86496 [details] [diff] [review]
Getting rid of NS_InitEmbedding2() per doug's comments above

sr=rpotts@netscape.com
Attachment #86496 - Flags: superreview+
(Assignee)

Comment 12

16 years ago
Landed patch on the trunk
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 13

16 years ago
Verified against Mozilla 1.1a Gecko20020617. MRE Component Reg is in
nsEmbedAPI.cpp. NS_InitEmbedding2 was removed.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.