247962 - Remove DLL_INitterm code from NSPR

Status: RESOLVED FIXED

(NSPR :: NSPR, defect)

Description

[Mike Kaply [:mkaply]]

Currently we mark ourselves as a PM app in the DLL_Initterm for NSPR.

This is wrong.

The app that is using NSPR might not be PM.

We should be marking ourselves as PM in Mozilla code.

This patch contains the NSPR change and the Mozilla change. I will be checking
the Mozilla change in immediately because it doesn't depend on the NSPR change.

Comment 1

[Mike Kaply [:mkaply]]

Attachment: Move Setting app to PM to Mozilla

Unfortunately I have to do this in three places.

Comment 2

[Mike Kaply [:mkaply]]

wtc, what is your opinion on this?

Comment 3

[jhp (no longer active)]

Comment on attachment 151368 [details] [diff] [review]
Move Setting app to PM to Mozilla

Looks good, but you'll also need to do this in mozilla/toolkit for ffox and
tbird (calender/nvu?).

Comment 4

[Julien Pierre]

FYI, command-line apps using NSPR, such as the NSS tools, already work without
this patch.

Comment 5

[Mike Kaply [:mkaply]]

wtc: could I get checkin authority to NSPR trunk? I would love to be able to fix
these types of Os/2 problems myself.

Thanks

Comment 6

[Wan-Teh Chang]

Comment on attachment 151368 [details] [diff] [review]
Move Setting app to PM to Mozilla

I verified that this patch only modifies OS/2 specific
code.  It was proposed by an OS/2 programmer and reviewed
by an OS/2 programmer. r=wtc.

Darin, could you check in at least the NSPR portion
of this patch?	Please check in the NSPR changes on
both the NSPR trunk and NSPRPUB_PRE_4_2_CLIENT_BRANCH.
Thanks.

Comment 7

[Mike Kaply [:mkaply]]

I checked this in on the client branch. Darin is going to put it on NSPR trunk.

Comment 8

[Darin Fisher]

I'll check this in on the trunk just as soon as my pull of the NSPR trunk
completes.  Sorry for the delay.

Comment 9

[Darin Fisher]

fixed on NSPR trunk