36974 - Bookmark should use nsIFile

Status: RESOLVED DUPLICATE of bug 191783

(SeaMonkey :: Bookmarks & History, defect, P5)

Description

[Doug Turner (:dougt)]

Bookmark should use nsIFile.  nsFileSpec is unsupported.


NTDLL! 77f7629c()
nsDebug::Assertion(const char * 0x100bb474, const char * 0x100bb470, const char 
* 0x100bb444, int 0x0000033c) line 191 + 13 bytes
nsFileSpec::nsFileSpec() line 828 + 34 bytes
nsBookmarksService::ReadBookmarks(nsBookmarksService * const 0x02999a50) line 
4106
nsBookmarksService::Init() line 1941 + 12 bytes
nsBookmarksServiceConstructor(nsISupports * 0x00000000, const nsID & {...}, void 
* * 0x0012efac) line 4841 + 146 bytes
nsGenericFactory::CreateInstance(nsGenericFactory * const 0x02999b50, 
nsISupports * 0x00000000, const nsID & {...}, void * * 0x0012efac) line 48
nsComponentManagerImpl::CreateInstance(nsComponentManagerImpl * const 
0x00c44370, const nsID & {...}, nsISupports * 0x00000000, const nsID & {...}, 
void * * 0x0012efac) line 1156 + 24 bytes

Comment 1

[Peter Trudelle]

[nsbeta2-], if renominated, please provide a user-level scenario showing what 
happens that is bad.

Comment 2

[Doug Turner (:dougt)]

Users will not be able to bookmark UNC paths.  

Comment 3

[bobj]

Does "UNC" mean non-ASCII?

Comment 4

[Doug Turner (:dougt)]

Universal Naming Convention:  It is a way to specify shared file objects:

\\server\volume\path\filename

In fact, if you bookmark a path like this, you will assert.

Comment 5

[leger]

Putting on [nsbeta2-] radar.  

Comment 6

[don]

Move to M20 target milestone.

Comment 7

[verah (gone)]

Nav triage team: [nsbeta3+]

Comment 8

[Daniel Veditz [:dveditz]]

Adding nsbeta3 keyword to bugs which already have nsbeta3 status markings so 
the queries don't get all screwed up.

Comment 9

[Phil Peterson]

PDT thinks this a P5. We can live without UNC bookmarks.

Comment 10

[Doug Turner (:dougt)]

In agreement.  We have marked other UNC related bugs Future.  Workaround is to
map the UNC path to a drive letter.

Comment 11

[Blake Ross]

renominating for beta3 per last two comments.  I don't think this is a beta3 
stopper.

Comment 12

[Claudius Gayle]

'renominating' makes it sound like this bug got minused are you are now advocating that it
be +'d. Viewing the bug activity has made me understand that instead you have "removed the
nsbeta3+ because you don't believe it to be worthy of that status and would like it to be
reconsidered."

Comment 13

[Steve Lamm]

Blake probably renominated this to change it from nsbeta3+ to nebeta3-.

Comment 14

[Steve Lamm]

46394 was marked nsbeta3-, so unless its status changes, this will not get fix 
either. I added some comments to that bug (owned by warren) to see if that is 
ok.

Comment 15

[Blake Ross]

Sorry for any ambiguity...I *don't* think this is a beta3 stopper, so I'd like 
it to be reconsidered.  Especially since it's dependant on a minused bug, as 
steve points out.

Comment 16

[verah (gone)]

Nav triage team: nsbeta3-  :(

Comment 17

[Blake Ross]

Reassigning 79 Bookmarks bugs to Ben.  I was told this was going to be done 
shortly about two months ago, but it clearly hasn't been.  I think that's long 
enough for all these bugs to remain assigned to nobody.

Feel free to filter all this spam into the trashcan by looking for this string 
in the message body: ducksgoquack

Comment 18

[Marcia Knous [:marcia]]

Netscape Nav triage team: this is not a Netscape beta stopper.

Comment 19

[Paul Chen]

Marking nsbeta1- bugs as future to get off the radar.

Comment 20

[Ben Goodger (use ben at mozilla dot org for email)]

Paul Chen is now taking Bookmarks bugs. For your convenience, you can filter 
email notifications caused by this by searching for 'ilikegoats'.

Comment 21

[Jaime Rodriguez, Jr.]

Removing adequated PDT grafitti.

Comment 22

[Peter Trudelle]

mass reassign of pchen bookmark bugs to ben

Comment 23

[Benjamin Smedberg]

*** This bug has been marked as a duplicate of 191783 ***