Spell Checker does not replace the wrong spelling with the right one

NEW
Unassigned

Status

()

Core
Spelling checker
4 years ago
4 years ago

People

(Reporter: Aniruddha, Unassigned)

Tracking

({regression})

6 Branch
x86_64
Windows 7
regression
Points:
---

Firefox Tracking Flags

(firefox25 affected, firefox26 affected, firefox27 affected, firefox28 affected, firefox-esr17 affected, firefox-esr24 affected)

Details

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
Created attachment 8341558 [details]
test.html

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20100101 Firefox/25.0 (Beta/Release)
Build ID: 20131112160018

Steps to reproduce:

Firefox 25.0.1
1. Open the attached html file
2. Right click on the incorrect spelling and try to correct it.


Actual results:

The wrong spelling does not get replaced with the right spelling. There is the following error in the JavaScript console:

[12:18:15.542] NS_ERROR_INVALID_POINTER: Component returned failure code: 0x80004003 (NS_ERROR_INVALID_POINTER) [nsIInlineSpellChecker.replaceWord] @ resource://gre/modules/InlineSpellChecker.jsm:273


NOTE: It works fine if I perform the entire operation without bringing the mouse pointer anywhere near the text area. i.e. Using the keyboard 
1. Tab key to tab into the textarea
2. Key to the left of the Ctrl key to bring up the context/popup menu
3. Up-down navigation keys to scroll through the spelling suggestions 
4. Enter key to select one of the suggestions. 
Then it works


Expected results:

The wrong spelling should get replaced with the right spelling

Updated

4 years ago
Status: UNCONFIRMED → NEW
Component: Untriaged → Spelling checker
Ever confirmed: true
Keywords: regression
Product: Firefox → Core
Version: 25 Branch → 6 Branch

Comment 1

4 years ago
Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/3589bf459f85
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0a1) Gecko/20110422 Firefox/6.0a1 ID:20110422183205
Bad:
http://hg.mozilla.org/mozilla-central/rev/b7b5af018848
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0a1) Gecko/20110422 Firefox/6.0a1 ID:20110422184500
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=3589bf459f85&tochange=b7b5af018848

Suspected :Bug 197686
Blocks: 197686

Updated

4 years ago
status-firefox25: --- → affected
status-firefox26: --- → affected
status-firefox27: --- → affected
status-firefox28: --- → affected
status-firefox-esr17: --- → affected
status-firefox-esr24: --- → affected
Presumably the same thing has always been broken in standards mode?

And even more presumably the issue is that the reframe caused by the hover style confuses the spellechecker UI somehow....

Comment 3

4 years ago
(In reply to Boris Zbarsky [:bz] from comment #2)
> Presumably the same thing has always been broken in standards mode?
> 
> And even more presumably the issue is that the reframe caused by the hover
> style confuses the spellechecker UI somehow....

Yes, that makes sense.

Aniruddha, is this issue breaking your website?  Can you please let me know what the impact of this bug is?

Thanks!
Flags: needinfo?(aniruddha.galgali)
(Reporter)

Comment 4

4 years ago
Well, it is a usability issue, not a very high impact, but nice to have a fix.
Flags: needinfo?(aniruddha.galgali)
(Reporter)

Comment 5

4 years ago
Any updates on this? When can we expect a fix?

Comment 6

4 years ago
Created attachment 8428393 [details] [diff] [review]
WIP

This patch fixes some of the mozInlineSpellChecker re-initialization bits so that it gets properly attached to the editor object, etc.  This eliminates the null pointer errors in the stderr spew but still doesn't fix this test case.  I don't have more time to look into why right now.
You need to log in before you can comment on or make changes to this bug.