Facebook comment box disables spell check

VERIFIED FIXED in Firefox 12

Status

()

Core
Editor
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: Terry R., Assigned: Ehsan)

Tracking

({regression})

10 Branch
mozilla14
x86
All
regression
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox12+ verified, firefox13+ verified)

Details

(Whiteboard: [qa+], URL)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Log into Facebook. Type a misspelled word into the "What's on your mind?" box, and spell check is functioning.  Enter a misspelled word in a Comment box.  Spell check is disabled.

Someone contacted Facebook, but they were told spell check was a browser feature, and not a Facebook issue.  Here is the link on the closed case.
http://developers.facebook.com/bugs/290435294356411/?browse=search_4f4cdc294c2bb2436078759
Dupe of bug 707434?
Status: NEW → UNCONFIRMED
Ever confirmed: false
(Reporter)

Comment 2

5 years ago
(In reply to Ryan VanderMeulen from comment #1)
> Dupe of bug 707434?

Ryan,

Doesn't the filing of this bug "confirm" bug 707434?  This occurs on any machine I've tested it on.
I was under the impression that disabling spell check was a purposeful decision on Facebook's part. Ehsan?
(Reporter)

Comment 4

5 years ago
(In reply to Ryan VanderMeulen from comment #3)
> I was under the impression that disabling spell check was a purposeful
> decision on Facebook's part. Ehsan?

Do you have any documentation to support that "decision"?

Comment 5

5 years ago
Seeing the same issue here (Firefox 11 on Windows, two different machines).  Also can reproduce in the box to reply to a private message thread on Facebook.

Interesting tidbit:  If I inspect the message box with Firebug, and then focus/unfocus it, spellcheck seems to kick in.
(In reply to Aaron Kelley from comment #5)
> Interesting tidbit:  If I inspect the message box with Firebug, and then
> focus/unfocus it, spellcheck seems to kick in.

OK, I see the problem now.  I'll take a look.
Assignee: nobody → ehsan
Blocks: 364914
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
The problem here is that the text box in question has an onfocus handler which causes a reframe which causes nsEditorEventListener::Focus to never be called.
This regression is visible on the highly popular site Facebook.  Nominating to track for Firefox 12 and 13.
tracking-firefox12: --- → ?
tracking-firefox13: --- → ?
Component: Untriaged → Editor
Keywords: regression
Product: Firefox → Core
QA Contact: untriaged → editor
Created attachment 606085 [details] [diff] [review]
Patch (v1)
Attachment #606085 - Flags: review?(roc)
Whiteboard: [autoland-try]

Updated

5 years ago
Whiteboard: [autoland-try] → [autoland-in-queue]

Comment 10

5 years ago
Autoland Patchset:
	Patches: 606085
	Branch: mozilla-central => try
	Destination: http://hg.mozilla.org/try/pushloghtml?changeset=7986be090ed0
Try run started, revision 7986be090ed0. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=7986be090ed0

Comment 11

5 years ago
Try run for 7986be090ed0 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=7986be090ed0
Results (out of 14 total builds):
    success: 14
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/autolanduser@mozilla.com-7986be090ed0

Updated

5 years ago
Whiteboard: [autoland-in-queue]
http://tbpl.mozilla.org/?tree=Try&rev=445c5a896588

Comment 13

5 years ago
Try run for 445c5a896588 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=445c5a896588
Results (out of 221 total builds):
    exception: 1
    success: 180
    warnings: 40
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/eakhgari@mozilla.com-445c5a896588
tracking-firefox12: ? → +
tracking-firefox13: ? → +
(Reporter)

Updated

5 years ago
OS: Windows XP → All
Attachment #606085 - Flags: review?(roc) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/fb904233c37c
Target Milestone: --- → mozilla14
Comment on attachment 606085 [details] [diff] [review]
Patch (v1)

[Approval Request Comment]
Regression caused by (bug #): bug 364914
User impact if declined: This bug is visible on facebook.com, which is pretty popular!  This bug makes it so that most of user's comments on Facebook would not be spell checked.
Testing completed (on m-c, etc.): I've tested this against the testcase I have manually constructed, and also against facebook.com.
Risk to taking this patch (and alternatives if risky): The problem is well understood here, I don't believe the patch to be very risky.  The downside of not taking it is that this fix will not reach our users until Firefox 14.  Note that we've broken this in 10, and 11 has already been shipped with this bug. :(
String changes made by this patch: none
Attachment #606085 - Flags: approval-mozilla-beta?
Attachment #606085 - Flags: approval-mozilla-aurora?
Flags: in-testsuite+
(Reporter)

Comment 16

5 years ago
Given the large amount of Facebook users (who hopefully mostly use Firefox), wouldn't this warrant a patch between versions, to correct this earlier?
(In reply to Terry R. from comment #16)
> Given the large amount of Facebook users (who hopefully mostly use Firefox),
> wouldn't this warrant a patch between versions, to correct this earlier?

Let's first see if we can get Aurora/Beta approval...  :-)
https://hg.mozilla.org/mozilla-central/rev/fb904233c37c
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Comment on attachment 606085 [details] [diff] [review]
Patch (v1)

[Triage Comment]
Approved for Aurora 13, but we'll discuss uplifting to Beta 12 at today's channel meeting. This isn't a very user-visible regression (a lack of spell checking) and has been around since FF10, so if there's any associated risk my intuition is for this to land first in 13. That being said, I hear Facebook is a popular site or something...
Attachment #606085 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment on attachment 606085 [details] [diff] [review]
Patch (v1)

[Triage Comment]
We discussed this in the channel meeting - we're not confident that the extra testing time on the smaller Aurora population will shake out any regressions. We think that the remaining 5 weeks in the beta cycle will. Approved for Beta 12 as well.
Attachment #606085 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Comment 21

5 years ago
The problem persists for Messages too... If you type misspell in messages firefox won't underline it
(In reply to bogas04 from comment #21)
> The problem persists for Messages too... If you type misspell in messages
> firefox won't underline it

Ouch, I filed bug 737889 for that.  :/
http://hg.mozilla.org/releases/mozilla-aurora/rev/158500e8fa4e
http://hg.mozilla.org/releases/mozilla-beta/rev/888db2929a6c
status-firefox12: --- → fixed
status-firefox13: --- → fixed
Whiteboard: [qa+]

Comment 24

5 years ago
Verified as fixed on:
Mozilla/5.0 (Windows NT 6.1; rv:12.0) Gecko/20100101 Firefox/12.0
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20100101 Firefox/12.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:12.0) Gecko/20100101 Firefox/12.0
20120328051619
status-firefox12: fixed → verified
Duplicate of this bug: 681559

Comment 26

5 years ago
I like how this is going, though I'm only skimming over the dev-talk. I hope to see this fixed soon, at least by Firefox 12!
I already told you that it's fixed for 12. But you don't even need to take my word for it. Betas of Firefox 12 are easily available for you to try for yourself.

Comment 28

5 years ago
Chill chill! I only just read your comments on the other bug!! Thanks for the info.

Comment 29

5 years ago
Verified as fixed on:
Mozilla/5.0 (X11; Linux i686; rv:13.0) Gecko/20100101 Firefox/13.0
Mozilla/5.0 (Windows NT 6.1; rv:13.0) Gecko/20100101 Firefox/13.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:13.0) Gecko/20100101 Firefox/13.0
BuildID: 20120425123149
Status: RESOLVED → VERIFIED
status-firefox13: fixed → verified
Duplicate of this bug: 756845
You need to log in before you can comment on or make changes to this bug.