Closed Bug 347819 Opened 14 years ago Closed 14 years ago
Change some spellchecker assertions to just return failure
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
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.