User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:57.0) Gecko/20100101 Firefox/57.0 Build ID: 20171019140425 Steps to reproduce: I copied a URL from a html file and pasted it in the /Show All bokomarks+right click+add new bookmark/ Then I clicked OK. And tht was that. It happened in version ~ 52 (i think --- around 6 months ago). Now i use version 58 (nightly) but have snyc enabled across all three distributions (nightly, dev and release) in win and linux. Actual results: Since then --- i cannot edit, copy, delete, cut or do anything with this bookmark ( i can still do everything with other bookmarks) --and the folders/subfolders in which this bookmark is located -- i can rename folder but not delete it. What is worse --- my firefox sync stopped working because of that... i have firefox sync enabled on three different desktop computers and now i see that bookmarks which i saved in one computer stayed only there -- didnt move everywhere like it should. Expected results: Even with this character in URL (although is invalid) the bookmark itself still should be editable/deletable etc... ps. to reproduce you can do the same, but warning.... once it is in, it doesnt want to come out again... so do it in an environment where there is no that many bookmarks (i have more than 10.000)
The promiseBookmarksTree code currently say we'll let NetUtil throw and the item will be skipped: http://searchfox.org/mozilla-central/rev/1285ae3e810e2dbf2652c49ee539e3199fcfe820/toolkit/components/places/PlacesUtils.jsm#1776 Note sure that's what we really want to do in this case...but not sure what impact it would have elsewhere. I also took a brief look at allowing editing of the bookmark, but this also ends up in similar issues where Bookmarks.jsm can't understand what is in the database and throws as a result of the bad URI. The only other though I had was a background task to clean up invalid URIs (i.e. probably remove them). I'm not quite sure what is best/simplest to do here.
(In reply to Mark Banner (:standard8) from comment #2) > The only other though I had was a background task to clean up invalid URIs > (i.e. probably remove them). We have various other bugs about similar issues, and it's likely a good idea, though, it may be expensive so we need a way to do that in chunks. I suppose we could use the place id to chunk and store it somewhere (a temp pref) then do the cleanup on idle timers. This may require a new approach to PlacesDBUtils to do cleanups in small chunks every day, vs doing a large weekly cleanup.
Priority: -- → P2
Since we don't allow anymore to create invalid bookmarks, what's left is being unable to get rid of them. So this is pretty much the same as bug 1401401.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1401401
You need to log in before you can comment on or make changes to this bug.