Closed Bug 330842 Opened 18 years ago Closed 18 years ago

"ASSERT: Can't serialize: file doesn't exist!" at startup

Categories

(Firefox :: Search, defect, P1)

2.0 Branch
x86
Linux
defect

Tracking

()

RESOLVED FIXED
Firefox 2 alpha2

People

(Reporter: lipsitt, Assigned: Gavin)

References

Details

(Keywords: fixed1.8.1, Whiteboard: [swag: 0d])

Attachments

(3 files, 1 obsolete file)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a1) Gecko/20060317 Firefox/1.6a1
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a1) Gecko/20060317 Firefox/1.6a1

I get an assertion failure window as soon as the browser starts. If I click OK a browser window opens. So far the browser appears to be functioning properly otherwise.

Here is the top of the message:

ASSERT: Can't serialize: file doesn't exist!
Stack Trace:
0:ENSURE_WARN(false,Can't serialize: file doesn't exist:, 2147549183)
1:SRCH_ENG_serializeToFile()
2:SRCH_SVC_convertSherlock([object Object],rpmfine)

If you wan't the rest of the stack trace let me know, but I have to type it in from a screenshot.

Reproducible: Always

Steps to Reproduce:
Start browser
Version: unspecified → Trunk
Component: General → Search
QA Contact: general → search
Do you get this every time you start up? Can you send me the contents of your profile's searchplugins directory?
You could also try setting the browser.search.log pref to true and checking STDOUT or the Javascript console for messages.
Daniel, are you still seeing this? If so, could you try some of the things mentioned in the previous comments?
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a1) Gecko/20060317 Firefox/1.6a1 ID:2006031704

I saw this message several times after updating but it does not appear any more.
Ah, so those attachments show that the initial conversion failed, but the renaming of the files still worked. Sounds like there is something wrong with my backup code on linux.

Flummox: Can you try renaming the .xml files you attached to .src, and running Firefox again? You should get the error messages again. The JavaScript console log from that run will show the actual bug and not the side effects, so if you could attach that I'd appreciate it.
Assignee: nobody → gavin.sharp
Actually, you don't need to test, I think I found the problem. The Linux nsIFile implementations seem to not change "this" after a moveTo or copyTo, while the Windows one does.
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Version: Trunk → 2.0 Branch
(which I've just realized is bug 200024)
Attached patch patch (obsolete) — Splinter Review
The first hunk is just code simplification.
Attachment #215730 - Flags: review?(mconnor)
Priority: -- → P1
Target Milestone: --- → Firefox 2 alpha2
Whiteboard: [patch-r?]
Comment on attachment 215730 [details] [diff] [review]
patch

>Index: browser/components/search/nsSearchService.js

>+    // Set aEngine._file explicitly, since some moveTo implementations don't
>+    // change srcFile (and thus aEngine._file) to point to the new file
>+    // (bug 200024)
>+    aEngine._file = xmlFile;

Actually, I'll remove this comment. Setting _file explicitly will always be correct, regardless of bug 200024's resolution. Even if nsLocalFileUnix is changed to match Windows, relying on that makes code hard to follow, so this comment would serve no purpose.
Attachment #215730 - Attachment filename: tmp.diff → 330842.diff
Comment on attachment 215730 [details] [diff] [review]
patch

bitrotted by the patch in bug 330887
Attachment #215730 - Attachment is obsolete: true
Attachment #215730 - Flags: review?(mconnor)
Whiteboard: [patch-r?]
Flags: blocking-firefox2+
Whiteboard: swag needed
Comment on attachment 215730 [details] [diff] [review]
patch

this still applies, and shouldn't be blocked by bug 330842
Attachment #215730 - Attachment is obsolete: false
Attachment #215730 - Flags: review?(mconnor)
Attachment #215730 - Flags: approval-branch-1.8.1?(mconnor)
Whiteboard: swag needed → [swag: 0d]
Comment on attachment 215730 [details] [diff] [review]
patch

Ok, though I don't see how oldFile/newFile is measurably improved by changing to srcFile/xmlFile.  how about originalSherlockFile and newSearchFile, in line with the recent discussions on better var names?
Attachment #215730 - Flags: review?(mconnor)
Attachment #215730 - Flags: review+
Attachment #215730 - Flags: approval-branch-1.8.1?(mconnor)
Attachment #215730 - Flags: approval-branch-1.8.1+
Attached patch for checkinSplinter Review
Attachment #215730 - Attachment is obsolete: true
checked in branch and trunk
mozilla/browser/components/search/nsSearchService.js 	1.1.2.8
mozilla/browser/components/search/nsSearchService.js 	1.9
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Checkin for this bug may have caused a 15% increase in number of allocations on balsa. See bug 333173.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: