Using nsFileSpec on Mac requires use of ResolveAlias(). To write XP code, this requires, e.g., nsFileSpec spec(...); spec += "subdir"; #ifdef XP_MAC { PRBool wasAlias; spec.ResolveAlias(wasAlias); } #endif ...which is exactly the kind of code we're trying to avoid. IMHO :). Shouldn't these be resolved automatically? (Hey, soft links on Unix are...)

Accepting for M7

Why is this assigned to me?

Because you own xpcom, and file i/o is a part of xpcom. Anyway, I fixed this today. Marking it fixed.

Did the ResolveAlias() method go away? Or is it still there, being called twice? In other words, will I need to go through and remove the #ifdef XP_MAC code at each call site that I alluded to above? thanks...

The "ResolveAlias()" method remains, and is called internally in a number of places now. After all outside calls have been removed, we might consider making this call protected, though. The only harm in calling it now is that it will be a NOOP that takes extra time that we can ill afford.

Bulk moving to XPCOM, in preparation for removal of XP File Handling component. (XPFH has received two bugs in ~5 months, and is no longer in active use.)