476230 - remove all MoreFilesX usage

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Josh Aas]

This bug is specifically about removing all MoreFilesX usage from the tree and removing the MoreFilesX component. This is part of bug 456662.

Comment 1

[Josh Aas]

Attachment: fix v1.0

Comment 2

[Steven Michaud [:smichaud] (Retired)]

Comment on attachment 359848 [details] [diff] [review]
fix v1.0

Aside from one minor mistake, this looks fine to me.

In nsLocalFile::GetFileCreator(), this:

+  id creatorNum = (NSNumber*)[dict objectForKey:NSFileHFSCreatorCode];

should really be this:

+  NSNumber* creatorNum = (NSNumber*)[dict objectForKey:NSFileHFSCreatorCode];

I also have one observation.  It's not an objection -- I just want to
note it for future reference.  (And I somehow missed it when reviewing
this patch's predecessor at bug 456662.)

nsLocalFile::InitWithFSSpec() and nsLocalFile::GetFSSpec()
(apparently) originally fell back to using the parent directory (or
trying to use it) on an error.  I'm not sure this ever made much sense
... but in any case you've gotten rid of it.

I compiled and tested this patch ... but I'm not entirely sure how it
should be tested.  I just ran the browser and opened a local file.
Everything worked fine.

Comment 3

[Josh Aas]

pushed to mozilla-central, forgot to make the change Steven suggested, will follow up

http://hg.mozilla.org/mozilla-central/rev/85e381233997

Comment 4

[Steven Michaud [:smichaud] (Retired)]

This should probably be backed out right away -- it appears to have caused bug 478549.