In embedding projects, add triggering PSM init during startup

VERIFIED FIXED in Future

Status

Core Graveyard
Security: UI
P3
normal
VERIFIED FIXED
16 years ago
a year ago

People

(Reporter: Kai Engert, Assigned: kaie)

Tracking

1.0 Branch
Future

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

16 years ago
See bug 76915 for the background. Basically, if there is the risk of having no
write access or a full disk, one should make sure during init that PSM can init
properly.

Some comments from an off-bugzilla discussion:


If we're initializing it explictly, and doing things to nsAppRunner.cpp, don't
forget about embedding. If the
initialization was removed from nsGlobalWindowImpl and into nsAppRunner.cpp,
embedding apps would be in a world
of hurt. Something similar would have to be done in NS_InitEmbedding().

Conrad Carlen wrote:
> 
> > Kai Engert wrote:
> >
> >> Regarding NS_InitEmbedding I'd be glad to get some help. Can you
> >> tell me
> >> at which place I should add the call, and what service/code should
> >> I
> >> call to display the error? Can I expect the error file to be
> >> available
> >> as it is in the non-embedding case, or do we need to do something
> >> completely different?
> >>
> This error is first detected in nsNSSComponent.cpp here:
> 
> +  if (NSS_InitReadWrite(profileStr) != SECSuccess) {
> 
> right? If that's true, the initialization and error check can't be in
> NS_InitEmbedding because, at that point, the profile has not been set.
> I think that embedding apps may have to handle this check themselves
> :-/
(Reporter)

Updated

16 years ago
Target Milestone: --- → Future
(Reporter)

Comment 1

16 years ago
Changing my prefered e-mail address.
Assignee: kai.engert → kaie

Comment 2

16 years ago
QA > junruh
Priority: -- → P3
QA Contact: ckritzer → junruh
Version: unspecified → 2.1
(Assignee)

Comment 3

16 years ago
If the patch for bug 75947 gets accepted, this bug will become obsolete.

Comment 4

16 years ago
Marking fixed per kaie's comment.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 5

16 years ago
Verified.
Status: RESOLVED → VERIFIED

Updated

13 years ago
Component: Security: UI → Security: UI
Product: PSM → Core

Updated

9 years ago
Version: psm2.1 → 1.0 Branch
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.