Closed Bug 347819 Opened 14 years ago Closed 14 years ago

Change some spellchecker assertions to just return failure

Categories

(Core :: Spelling checker, defect)

1.8 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: brettw, Assigned: brettw)

References

Details

(Keywords: fixed1.8.1)

Attachments

(1 file)

 
Attachment #232641 - Flags: review?(bryner)
here are the assertions I'm seeing after the fix for bug #347561:

WARNING: NS_ENSURE_TRUE(editor) failed, file
c:/builds/bonecho/mozilla/extension
s/spellcheck/src/mozInlineSpellChecker.cpp, line 1125

WARNING: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed, file
c:/builds/bonecho/mozilla
/extensions/spellcheck/src/mozInlineSpellWordUtil.cpp, line 91

I'll get some stacks for brett.
Attachment #232641 - Flags: review?(bryner) → review+
Comment on attachment 232641 [details] [diff] [review]
Patch for the common cases

Requesting approval for this trivial change to assertions.
Attachment #232641 - Flags: approval1.8.1?
Fixed on trunk, leaving open for branch.
This patch also should improve performance a bit, since it saves a QueryReferent for every word checked.
Comment on attachment 232641 [details] [diff] [review]
Patch for the common cases

a=drivers, please land on the branch.
Attachment #232641 - Flags: approval1.8.1? → approval1.8.1+
Fixed on branch.
Status: NEW → RESOLVED
Closed: 14 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
> I'll get some stacks for brett.

brett, I've updated and rebuilt the branch on windows xp, and with your changes and I no longer assert.

thanks for fixing this.

You need to log in before you can comment on or make changes to this bug.