WebCore:
authorsullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 31 Oct 2006 02:54:41 +0000 (02:54 +0000)
committersullivan <sullivan@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 31 Oct 2006 02:54:41 +0000 (02:54 +0000)
        Reviewed by Geoff Garen

        WebCore part of change to push the code that updates the spelling panel
        into WebCore, in preparation for some grammar-checking stuff.

        * bridge/mac/FrameMac.h:
        * bridge/mac/FrameMac.mm:
        (WebCore::FrameMac::advanceToNextMisspelling):
        no more return value for this method, and update the spelling panel with
        the misspelled word here rather than in the WebKit callers.

WebKit:

        Reviewed by Geoff Garen

        * WebView/WebHTMLView.m:
        (-[NSArray checkSpelling:]):
        removed code to update spelling panel; WebCore handles that now
        (-[NSArray showGuessPanel:]):
        ditto

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@17464 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/bridge/mac/FrameMac.h
WebCore/bridge/mac/FrameMac.mm
WebKit/ChangeLog
WebKit/WebView/WebHTMLView.m

index 99538cf4f51662d83a7115cd86ed6d300917e5fa..9557f5be65e5d8b156110d7a850a8b424bc789e5 100644 (file)
@@ -1,3 +1,16 @@
+2006-10-30  John Sullivan  <sullivan@apple.com>
+
+        Reviewed by Geoff Garen
+        
+        WebCore part of change to push the code that updates the spelling panel
+        into WebCore, in preparation for some grammar-checking stuff.
+
+        * bridge/mac/FrameMac.h:
+        * bridge/mac/FrameMac.mm:
+        (WebCore::FrameMac::advanceToNextMisspelling):
+        no more return value for this method, and update the spelling panel with
+        the misspelled word here rather than in the WebKit callers.
+
 2006-10-30  Oliver Hunt  <oliver@apple.com>
 
         Reviewed by Anders.
index 6d684f55c1f0404896a06f103f3825aa816622ee..73cc0e39a2199e01c2b718ac6614ab3a178fbcbd 100644 (file)
@@ -112,7 +112,7 @@ public:
     virtual void setView(FrameView*);
     virtual void frameDetached();
 
-    String advanceToNextMisspelling(bool startBeforeSelection = false);
+    void advanceToNextMisspelling(bool startBeforeSelection = false);
     
     virtual void setTitle(const String&);
     virtual void setStatusBarText(const String&);
index 00dc5e063a61bf22aa0182b5e4db17cb0fe881f0..c8c64cc0795acf949b600596ada47fd488d4bb35 100644 (file)
@@ -565,7 +565,7 @@ void FrameMac::unfocusWindow()
     END_BLOCK_OBJC_EXCEPTIONS;
 }
 
-String FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
+void FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
 {
     int exception = 0;
 
@@ -592,9 +592,9 @@ String FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
     Node *editableNode = searchRange->startContainer(exception);
     if (!editableNode->isContentEditable()) {
         editableNode = editableNode->nextEditable();
-        if (!editableNode) {
-            return String();
-        }
+        if (!editableNode)
+            return;
+
         searchRange->setStartBefore(editableNode, exception);
         startedWithSelection = false;   // won't need to wrap
     }
@@ -613,12 +613,12 @@ String FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
     }
     
     if (searchRange->collapsed(exception))
-        return String();       // nothing to search in
+        return;       // nothing to search in
     
     // Get the spell checker if it is available
     NSSpellChecker *checker = [NSSpellChecker sharedSpellChecker];
     if (checker == nil)
-        return String();
+        return;
         
     WordAwareIterator it(searchRange.get());
     bool wrapped = false;
@@ -651,7 +651,8 @@ String FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
                     revealSelection();
                     // Mark misspelling in document.
                     document()->addMarker(misspellingRange.get(), DocumentMarker::Spelling);
-                    return result;
+                    [checker updateSpellingPanelWithMisspelledWord:result];
+                    return;
                 }
             }
         
@@ -671,7 +672,7 @@ String FrameMac::advanceToNextMisspelling(bool startBeforeSelection)
         }   
     }
     
-    return String();
+    return;
 }
 
 bool FrameMac::wheelEvent(NSEvent *event)
index ee0270f2fa1df0a461e9797f728c1b826e20916f..5c83ba186fe50a3748568faf85068e84a0c5e228 100644 (file)
@@ -1,3 +1,13 @@
+2006-10-30  John Sullivan  <sullivan@apple.com>
+
+        Reviewed by Geoff Garen
+
+        * WebView/WebHTMLView.m:
+        (-[NSArray checkSpelling:]):
+        removed code to update spelling panel; WebCore handles that now
+        (-[NSArray showGuessPanel:]):
+        ditto
+
 2006-10-30  John Sullivan  <sullivan@apple.com>
 
         * English.lproj/WebViewEditingContextMenu.nib/info.nib:
index 6301a26bf91914ce9f2dd01c38307710b019d9a9..92fc95c233b6b1d3fb804c119b24a5569f550b96 100644 (file)
@@ -4537,21 +4537,17 @@ NSStrokeColorAttributeName        /* NSColor, default nil: same as foreground co
 
 - (void)checkSpelling:(id)sender
 {
-    // WebCore does everything but update the spelling panel
     NSSpellChecker *checker = [NSSpellChecker sharedSpellChecker];
     if (!checker) {
         LOG_ERROR("No NSSpellChecker");
         return;
     }
     
-    NSString *badWord = core([self _frame])->advanceToNextMisspelling();
-    if (badWord)
-        [checker updateSpellingPanelWithMisspelledWord:badWord];
+    core([self _frame])->advanceToNextMisspelling();
 }
 
 - (void)showGuessPanel:(id)sender
 {
-    // WebCore does everything but update the spelling panel
     NSSpellChecker *checker = [NSSpellChecker sharedSpellChecker];
     if (!checker) {
         LOG_ERROR("No NSSpellChecker");
@@ -4568,9 +4564,7 @@ NSStrokeColorAttributeName        /* NSColor, default nil: same as foreground co
     }
 #endif
     
-    NSString *badWord = core([self _frame])->advanceToNextMisspelling(true);
-    if (badWord)
-        [checker updateSpellingPanelWithMisspelledWord:badWord];
+    core([self _frame])->advanceToNextMisspelling(true);
     [spellingPanel orderFront:sender];
 }