Closed
Bug 58231
Opened 25 years ago
Closed 25 years ago
[Mac] salt needed in just one more place.
Categories
(Core Graveyard :: Profile: BackEnd, defect, P3)
Tracking
(Not tracked)
People
(Reporter: racham, Assigned: sspitzer)
References
Details
Seth,
If the user deletes a parent profile folder, Mac does the resolution of that
situation much before other platforms (see bugs 56041 and 57361) and it creates
a directory so that all persistent operations get resolved properly. That
directory is empty to start with and it will filled, if the user chooses to run
the app with that profile. So, those situations take route of
PopulateIfEmptyDir() routine. If we identify a dir not having contents at that
time, we just copy all default contents there. So, we need the fix you did at
http://lxr.mozilla.org/seamonkey/source/profile/src/nsProfile.cpp#886 through
#889 at http://lxr.mozilla.org/seamonkey/source/profile/src/nsProfile.cpp#2294.
Sorry to bring up this late. I was doing more testing and came across this one.
Comment 1•25 years ago
|
||
Shouldn't this just be part of 56002? It's not checked in on the branch yet.
Blocks: 56002
I agree. May be there isno need for another bug here as we are just talking
about a case that's missed in bug 56602. Also, I think it's much easier even for
PDT to have all these issue a under one bug (infact it a single issue and that
is 'salting'). OK.. I moving the description for this case into bug 56002 and
close this a dup of that.
*** This bug has been marked as a duplicate of 56002 ***
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → DUPLICATE
Comment 4•25 years ago
|
||
Seth, I'm working on the profile code right now - fixing the fallout from 56002.
I have your salting code in place and the case Bhuvan mentions will be taken
care of. It probably already is.
Comment 5•25 years ago
|
||
Whoops, sorry. Not fallout from 56002 - I meant 56041.
Assignee | ||
Comment 6•25 years ago
|
||
actually, this only affects the trunk. this additional place is on new code
that isn't on the branch.
but we need the fix this on the trunk.
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•