make NS_MAC_DEFAULT_DOWNLOAD_DIR available for XP_MACOSX

VERIFIED FIXED in Chimera0.5

Status

Camino Graveyard
OS Integration
VERIFIED FIXED
16 years ago
16 years ago

People

(Reporter: Steve Dagley, Assigned: Steve Dagley)

Tracking

unspecified
Chimera0.5
PowerPC
Mac OS X

Details

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
Ok, this isn't actually a Chimera specific bug but it's blocking a Chimera bug...

We need NS_MAC_DEFAULT_DOWNLOAD_DIR (corresponding to the user's specified
Download folder) under XP_MACOSX.  Not having it blocks #151047
(Assignee)

Updated

16 years ago
Blocks: 151047
OK
Status: NEW → ASSIGNED
Target Milestone: --- → Chimera0.5
Created attachment 95965 [details] [diff] [review]
patch

Defines NS_MAC_DEFAULT_DOWNLOAD_DIR, defines the home dir the right way, and
removes definition of some stuff that never should have been defined for
XP_MACOSX.

Looking for review.
(Assignee)

Comment 3

16 years ago
Comment on attachment 95965 [details] [diff] [review]
patch

r=sdagley (gee that code looks familiar :-)
Attachment #95965 - Flags: review+
(Assignee)

Comment 4

16 years ago
taking (since this will be landed as part of fix for #151047)
Assignee: ccarlen → sdagley
Status: ASSIGNED → NEW
(Assignee)

Comment 5

16 years ago
patch checed in by sfraser as part of checkin for #151047
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 6

16 years ago
Steve,

What's the best way to verify this ? Do I need to specify a new default path via
System Preferences- Internet ? Then Chimera will automatically recognize it when
saving a file ?
(Assignee)

Comment 7

16 years ago
Well at this point the only way to verify this is to eyeball the code is there.
 sfraser didn't check in all my patch for #151047 so this code currently isn't
being exercised (AFAIK)

Comment 8

16 years ago
Hmmm. Maybe I should wait for the rest of code to be checked in to really verify
it. I love to see this working.

Comment 9

16 years ago
Verified in the 2002-11-12-04 NB under 10.2.2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.