From 992aec0bc99a3a07a97f58ecdcd056baa11baee5 Mon Sep 17 00:00:00 2001
From: "bzbarsky%mit.edu"
 <bzbarsky%mit.edu@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Thu, 9 Feb 2006 03:50:53 +0000
Subject: [PATCH] Make it possible to enable spell-check for text inputs too
 (not just textareas).  Make the pref controlling the whole thing live.  Bug
 151040, patch by Julian Pellico <jpellico@yahoo.com>, r=bzbarsky, sr=bryner

git-svn-id: svn://10.0.0.236/trunk@189459 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/browser/app/profile/firefox.js        |  3 +-
 .../spellcheck/src/mozEnglishWordUtils.cpp    |  5 +++
 .../spellcheck/src/mozInlineSpellChecker.h    |  1 +
 mozilla/layout/forms/nsTextControlFrame.cpp   | 32 ++++++++++++++++---
 mozilla/layout/forms/nsTextControlFrame.h     |  8 +++++
 5 files changed, 43 insertions(+), 6 deletions(-)

diff --git a/mozilla/browser/app/profile/firefox.js b/mozilla/browser/app/profile/firefox.js
index a9ddc9055e6..fafed587125 100644
--- a/mozilla/browser/app/profile/firefox.js
+++ b/mozilla/browser/app/profile/firefox.js
@@ -434,7 +434,8 @@ pref("browser.backspace_action", 0);
 
 // this will automatically enable inline spellchecking (if it is available) for
 // multi-line text entry controls <textarea>s in HTML
-pref("layout.textarea.spellcheckDefault", true);
+// 0 = spellcheck nothing, 1 = check multi-line controls, 2 = check multi/single line controls
+pref("layout.spellcheckDefault", 1);
 
 pref("view_source.editor.path", "");
 pref("view_source.editor.external", false);
diff --git a/mozilla/extensions/spellcheck/src/mozEnglishWordUtils.cpp b/mozilla/extensions/spellcheck/src/mozEnglishWordUtils.cpp
index fecada5dbab..717b4619795 100644
--- a/mozilla/extensions/spellcheck/src/mozEnglishWordUtils.cpp
+++ b/mozilla/extensions/spellcheck/src/mozEnglishWordUtils.cpp
@@ -175,6 +175,11 @@ NS_IMETHODIMP mozEnglishWordUtils::FindNextWord(const PRUnichar *word, PRUint32
   const PRUnichar *startWord=p;
   if(p<endbuf){
     // XXX These loops should be modified to handle non-BMP characters.
+    // if previous character is a word character, need to advance out of the word
+    if (offset > 0 && ucIsAlpha(*(p-1))) {
+      while (p < endbuf && ucIsAlpha(*p))
+        p++;
+    }
     while((p < endbuf) && (!ucIsAlpha(*p)))
       {
         p++;
diff --git a/mozilla/extensions/spellcheck/src/mozInlineSpellChecker.h b/mozilla/extensions/spellcheck/src/mozInlineSpellChecker.h
index cb77378f12a..f981922b3d6 100644
--- a/mozilla/extensions/spellcheck/src/mozInlineSpellChecker.h
+++ b/mozilla/extensions/spellcheck/src/mozInlineSpellChecker.h
@@ -48,6 +48,7 @@
 #include "nsIDOMMouseListener.h"
 #include "nsIDOMKeyListener.h"
 #include "nsWeakReference.h"
+#include "mozISpellI18NUtil.h"
 
 class nsIDOMMouseEventListener;
 
diff --git a/mozilla/layout/forms/nsTextControlFrame.cpp b/mozilla/layout/forms/nsTextControlFrame.cpp
index 2347804a14c..ef24bce2140 100644
--- a/mozilla/layout/forms/nsTextControlFrame.cpp
+++ b/mozilla/layout/forms/nsTextControlFrame.cpp
@@ -133,7 +133,7 @@
 
 #define DEFAULT_COLUMN_WIDTH 20
 
-#define PREF_DEFAULT_SPELLCHECK "layout.textarea.spellcheckDefault"
+#define PREF_DEFAULT_SPELLCHECK "layout.spellcheckDefault"
 
 #include "nsContentCID.h"
 static NS_DEFINE_IID(kRangeCID,     NS_RANGE_CID);
@@ -1422,6 +1422,8 @@ nsTextControlFrame::PreDestroy(nsPresContext* aPresContext)
 NS_IMETHODIMP 
 nsTextControlFrame::Destroy(nsPresContext* aPresContext)
 {
+  nsContentUtils::UnregisterPrefCallback(PREF_DEFAULT_SPELLCHECK,
+                                         nsTextControlFrame::RealTimeSpellCallback, this);
   if (!mDidPreDestroy) {
     PreDestroy(aPresContext);
   }
@@ -1705,14 +1707,32 @@ nsTextControlFrame::SyncRealTimeSpell()
   }
 
   PRBool enable = PR_FALSE;
-  if (!readOnly && !IsSingleLineTextControl()) {
-    // multi-line text control: check the pref to see what the default should be
-    // GetBoolPref defaults the value to PR_FALSE is the pref is not set
-    enable = nsContentUtils::GetBoolPref(PREF_DEFAULT_SPELLCHECK);
+  if (!readOnly) {
+    // check the pref to see what the default should be, default is 0: never spellcheck
+    PRInt32 spellcheckLevel = nsContentUtils::GetIntPref(PREF_DEFAULT_SPELLCHECK, 0);
+    switch (spellcheckLevel) {
+      case SpellcheckAllTextFields:
+        enable = PR_TRUE;
+        break;
+      case SpellcheckMultiLineOnly:
+        enable = !IsSingleLineTextControl();
+        break;
+    }
   }
   SetEnableRealTimeSpell(enable);
 }
 
+// PrefCallback for real time spell pref
+// static
+int PR_CALLBACK nsTextControlFrame::RealTimeSpellCallback(const char* aPref, void* aContext)
+{
+  if (strcmp(aPref, PREF_DEFAULT_SPELLCHECK) == 0) {
+    nsTextControlFrame* frame = NS_STATIC_CAST(nsTextControlFrame*, aContext);
+    NS_ASSERTION(frame, "Pref callback: aContext was of an unexpected type");
+    frame->SyncRealTimeSpell();
+  }
+  return 0;
+}
 
 nsresult
 nsTextControlFrame::InitEditor()
@@ -1799,6 +1819,8 @@ nsTextControlFrame::InitEditor()
   transMgr->SetMaxTransactionCount(DEFAULT_UNDO_CAP);
 
   SyncRealTimeSpell();
+  nsContentUtils::RegisterPrefCallback(PREF_DEFAULT_SPELLCHECK, 
+                                       nsTextControlFrame::RealTimeSpellCallback, this);
 
   if (IsPasswordTextControl()) {
     // Disable undo for password textfields.  Note that we want to do this at
diff --git a/mozilla/layout/forms/nsTextControlFrame.h b/mozilla/layout/forms/nsTextControlFrame.h
index e1a0aa9795b..7a602d4cc30 100644
--- a/mozilla/layout/forms/nsTextControlFrame.h
+++ b/mozilla/layout/forms/nsTextControlFrame.h
@@ -50,6 +50,7 @@
 #include "nsWeakReference.h" //for service and presshell pointers
 #include "nsIScrollableViewProvider.h"
 #include "nsIPhonetic.h"
+#include "nsContentUtils.h"
 
 class nsISupportsArray;
 class nsIEditor;
@@ -184,6 +185,12 @@ public: //for methods who access nsTextControlFrame directly
 
   /* called to free up native keybinding services */
   static NS_HIDDEN_(void) ShutDown();
+  
+  enum SpellcheckDefaultState {
+    SpellcheckNone = 0,
+    SpellcheckMultiLineOnly = 1,
+    SpellcheckAllTextFields = 2
+  };
 
 protected:
   /**
@@ -265,6 +272,7 @@ private:
   
   void SetEnableRealTimeSpell(PRBool aEnabled);
   void SyncRealTimeSpell();
+  static int PR_CALLBACK RealTimeSpellCallback(const char* aPref, void* aContext);
 
 private:
   nsCOMPtr<nsIEditor> mEditor;