User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:126.96.36.199) Gecko/20060911 Camino/1.0.3 (MultiLang) Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:188.8.131.52) Gecko/20060911 Camino/1.0.3 (MultiLang) I drag this link on camino, and it downloads itself (the .URL bookmark) again and again. Camino is unusable ans my downloads folder is overhelmed with this auto-duplicating file. Safari does well with the .URL bookmark. Reproducible: Always Steps to Reproduce: 1.drag the .URL to camino 2.Let it bug 3.Force to quit Expected Results: open the embedded url of the .URL file I wish I could upload you a .URL bookmark for you to investigate on this issue.
the .URL file, opened with textedit is (for example): [InternetShortcut] URL=http://www.laserpod.com/introduction.asp and the filename (open with camino by default)is: laserpod.URL
Change the extension from .URL to .url and it will work. However, I should probably make the extension check case-insensitive. Taking.
Assignee: nobody → mozilla
Status: UNCONFIRMED → NEW
Ever confirmed: true
The fix is to lowercase the file extension before the isEqualToString: calls.
Severity -> normal. I don't see any auto-duplication, and the fix is trivial, so...
Severity: critical → normal
Oops, I missed the one in MainController.mm.
r+ as per froodian on IRC.
Fixing up the BM manager too, so we don't have to worry about it later (per smorgan).
Attachment #240835 - Attachment is obsolete: true
Comment on attachment 240841 [details] [diff] [review] v3: Add BookmarkManager.mm, too. r=me
Attachment #240841 - Flags: superreview?(sfraser_bugs) → superreview+
This looks simple enough to consider for 1.0.4 when the time comes.
Whiteboard: [needs checkin]
Checked in on trunk and 1.8branch
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Whiteboard: [needs checkin]
This patch fails to apply on the 1.8.0 branch. If someone whips up a 180branch-compatible patch, we'll consider this for 1.0.4.
I'll have to check out that branch, but I should be able to do it.
Comment on attachment 248433 [details] [diff] [review] Patch for 1.8.0 branch I tested this, and it works as expected (within the confines of bug 363654 on the 1.8.0 branch and bug 363633 everywhere).
Attachment #248433 - Flags: review+
Flags: camino1.0.4? → camino1.0.4+
Comment on attachment 248433 [details] [diff] [review] Patch for 1.8.0 branch r=me. Since the differences are redonkulous, foregoing a second sr. If anybody disagrees, they should speak up before I get to checkins.
Attachment #248433 - Flags: review?(stridey) → review+
Checked in on 1.8.0branch (forgot bug number in checkin comment, search for "robustification")
Whiteboard: [Needs Checkin 1.8.0branch]
You need to log in before you can comment on or make changes to this bug.