LayoutTests:
authorjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jul 2006 02:21:00 +0000 (02:21 +0000)
committerjusting <justing@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jul 2006 02:21:00 +0000 (02:21 +0000)
        Reviewed by levi

        <rdar://problem/4620686>
        REGRESSION: Mispelling markers are still displayed when using the delete key to place caret into a misspelled word

        * editing/deleting/delete-after-span-ws-001-expected.checksum:
        * editing/deleting/delete-after-span-ws-001-expected.png:
        * editing/deleting/delete-after-span-ws-002-expected.checksum:
        * editing/deleting/delete-after-span-ws-002-expected.png:
        * editing/deleting/delete-after-span-ws-003-expected.checksum:
        * editing/deleting/delete-after-span-ws-003-expected.png:
        * editing/deleting/delete-block-merge-contents-010-expected.checksum:
        * editing/deleting/delete-block-merge-contents-010-expected.png:
        * editing/deleting/merge-unrendered-space-expected.checksum:
        * editing/deleting/merge-unrendered-space-expected.png:

WebCore:

        Reviewed by levi

        <rdar://problem/4620686>
        REGRESSION: Mispelling markers are still displayed when using the delete key to place caret into a misspelled word

        * bridge/mac/FrameMac.mm:
        (WebCore::FrameMac::respondToChangedSelection): Remove markers from newAdjacentWords
        even if oldAdjacentWords is equal to newAdjacentWords.  This happens during a deletion.

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

14 files changed:
LayoutTests/ChangeLog
LayoutTests/editing/deleting/delete-after-span-ws-001-expected.checksum
LayoutTests/editing/deleting/delete-after-span-ws-001-expected.png
LayoutTests/editing/deleting/delete-after-span-ws-002-expected.checksum
LayoutTests/editing/deleting/delete-after-span-ws-002-expected.png
LayoutTests/editing/deleting/delete-after-span-ws-003-expected.checksum
LayoutTests/editing/deleting/delete-after-span-ws-003-expected.png
LayoutTests/editing/deleting/delete-block-merge-contents-010-expected.checksum
LayoutTests/editing/deleting/delete-block-merge-contents-010-expected.png
LayoutTests/editing/deleting/merge-unrendered-space-expected.checksum
LayoutTests/editing/deleting/merge-unrendered-space-expected.png
WebCore/ChangeLog
WebCore/WebCore.xcodeproj/project.pbxproj
WebCore/bridge/mac/FrameMac.mm

index ddb777f368232184d99a06a75f0579528480ccff..bcf2900344e499595e4dd7f40ce472c31eb7d15c 100644 (file)
@@ -1,3 +1,21 @@
+2006-07-11  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by levi
+        
+        <rdar://problem/4620686>
+        REGRESSION: Mispelling markers are still displayed when using the delete key to place caret into a misspelled word
+
+        * editing/deleting/delete-after-span-ws-001-expected.checksum:
+        * editing/deleting/delete-after-span-ws-001-expected.png:
+        * editing/deleting/delete-after-span-ws-002-expected.checksum:
+        * editing/deleting/delete-after-span-ws-002-expected.png:
+        * editing/deleting/delete-after-span-ws-003-expected.checksum:
+        * editing/deleting/delete-after-span-ws-003-expected.png:
+        * editing/deleting/delete-block-merge-contents-010-expected.checksum:
+        * editing/deleting/delete-block-merge-contents-010-expected.png:
+        * editing/deleting/merge-unrendered-space-expected.checksum:
+        * editing/deleting/merge-unrendered-space-expected.png:
+        
 2006-07-11  Tim Omernick  <timo@apple.com>
 
         Reviewed by Geoff.
index ae7d9bb4ab7e7888f01e141a024d68da0df5fada..95755ed6b6e2d81329f62523b61b37457264fafe 100644 (file)
@@ -1 +1 @@
-fa737256f176829ea9cd740df331f0f4
\ No newline at end of file
+db707667cfcac13812fe2d5c45f08c34
\ No newline at end of file
index f59c243825a09dce10617e1d66d178d086c84297..9b138c535f44a4f3cb687242cda4a68dc787dc5e 100644 (file)
Binary files a/LayoutTests/editing/deleting/delete-after-span-ws-001-expected.png and b/LayoutTests/editing/deleting/delete-after-span-ws-001-expected.png differ
index ae7d9bb4ab7e7888f01e141a024d68da0df5fada..95755ed6b6e2d81329f62523b61b37457264fafe 100644 (file)
@@ -1 +1 @@
-fa737256f176829ea9cd740df331f0f4
\ No newline at end of file
+db707667cfcac13812fe2d5c45f08c34
\ No newline at end of file
index f59c243825a09dce10617e1d66d178d086c84297..9b138c535f44a4f3cb687242cda4a68dc787dc5e 100644 (file)
Binary files a/LayoutTests/editing/deleting/delete-after-span-ws-002-expected.png and b/LayoutTests/editing/deleting/delete-after-span-ws-002-expected.png differ
index ae7d9bb4ab7e7888f01e141a024d68da0df5fada..95755ed6b6e2d81329f62523b61b37457264fafe 100644 (file)
@@ -1 +1 @@
-fa737256f176829ea9cd740df331f0f4
\ No newline at end of file
+db707667cfcac13812fe2d5c45f08c34
\ No newline at end of file
index f59c243825a09dce10617e1d66d178d086c84297..9b138c535f44a4f3cb687242cda4a68dc787dc5e 100644 (file)
Binary files a/LayoutTests/editing/deleting/delete-after-span-ws-003-expected.png and b/LayoutTests/editing/deleting/delete-after-span-ws-003-expected.png differ
index a27a528b9e82e0230cac3632d678da75593acce0..10e7acc0b7e9b2f4171ba5ed09d558e41299b0c5 100644 (file)
@@ -1 +1 @@
-811ab335e3a2365bca81e09645759b09
\ No newline at end of file
+0f40969b8062d08c9d8499295b0f5e9c
\ No newline at end of file
index ede87742321f8c77bfc11063361bd8d350a71010..108f644197c8f5930af9dc9ade623d7223b64db8 100644 (file)
Binary files a/LayoutTests/editing/deleting/delete-block-merge-contents-010-expected.png and b/LayoutTests/editing/deleting/delete-block-merge-contents-010-expected.png differ
index 327e03f45da5b3a4233365e5b86feadd31aa9f60..24685efdac07c8de82dd1b84fa56bc6056f5db74 100644 (file)
@@ -1 +1 @@
-330385e63988b427faaae197a4a81494
\ No newline at end of file
+c211a9ce650e8735f492839a38d55eb3
\ No newline at end of file
index 4725f015eddb15f49017281fd57db043eaa79036..ce7936d2ac2339076b144e8b44b0ab1226c43308 100644 (file)
Binary files a/LayoutTests/editing/deleting/merge-unrendered-space-expected.png and b/LayoutTests/editing/deleting/merge-unrendered-space-expected.png differ
index 49ec525d557025fa26ad6966f6899efba09bd876..34a646aad0969f42f1ec4cbe06bb4dceb00325dd 100644 (file)
@@ -1,3 +1,15 @@
+2006-07-11  Justin Garcia  <justin.garcia@apple.com>
+
+        Reviewed by levi
+        
+        <rdar://problem/4620686>
+        REGRESSION: Mispelling markers are still displayed when using the delete key to place caret into a misspelled word
+
+        * bridge/mac/FrameMac.mm:
+        (WebCore::FrameMac::respondToChangedSelection): Remove markers from newAdjacentWords 
+        even if oldAdjacentWords is equal to newAdjacentWords.  This happens during a deletion.
+        
+
 2006-07-11  Tim Omernick  <timo@apple.com>
 
         Reviewed by NOBODY (build fix)
index 2e82e1a11d6dcbe13cf1369dd910b79d0393d3ba..9d95619b99892883010303223661aca5867995db 100644 (file)
                FAE04190097596C9000540BE /* SVGImageLoader.h in Headers */ = {isa = PBXBuildFile; fileRef = FAE0418E097596C9000540BE /* SVGImageLoader.h */; };
 /* End PBXBuildFile section */
 
+/* Begin PBXBuildStyle section */
+               D098D1C70A6490E100D1917D /* Development */ = {
+                       isa = PBXBuildStyle;
+                       buildSettings = {
+                               COPY_PHASE_STRIP = NO;
+                       };
+                       name = Development;
+               };
+               D098D1C80A6490E100D1917D /* Deployment */ = {
+                       isa = PBXBuildStyle;
+                       buildSettings = {
+                               COPY_PHASE_STRIP = YES;
+                       };
+                       name = Deployment;
+               };
+/* End PBXBuildStyle section */
+
 /* Begin PBXContainerItemProxy section */
                DD041FF009D9E3250010AF2A /* PBXContainerItemProxy */ = {
                        isa = PBXContainerItemProxy;
                0867D690FE84028FC02AAC07 /* Project object */ = {
                        isa = PBXProject;
                        buildConfigurationList = 149C284308902B11008A9EFC /* Build configuration list for PBXProject "WebCore" */;
+                       buildSettings = {
+                       };
+                       buildStyles = (
+                               D098D1C70A6490E100D1917D /* Development */,
+                               D098D1C80A6490E100D1917D /* Deployment */,
+                       );
                        hasScannedForEncodings = 1;
                        knownRegions = (
                                English,
index 4bc55b9e7a6c8210ca845c6db7e884b420f11946..97343105856deb45f66d683eeaa8a946ac072177 100644 (file)
@@ -3143,22 +3143,13 @@ void FrameMac::respondToChangedSelection(const SelectionController &oldSelection
             VisiblePosition newStart(selection().start(), selection().affinity());
             SelectionController newAdjacentWords(startOfWord(newStart, LeftWordIfOnBoundary), endOfWord(newStart, RightWordIfOnBoundary));
 
-            if (oldAdjacentWords != newAdjacentWords) {
-                // Mark misspellings in the portion that was previously unmarked because of
-                // the proximity of the start of the selection. We only spell check words in
-                // the vicinity of the start of the old selection because the spelling checker
-                // is not fast enough to do a lot of spelling checking implicitly. This matches
-                // AppKit. This function is really the only code that knows that rule. The
-                // markMisspellings function is prepared to handler larger ranges.
-
-                // When typing we check spelling elsewhere, so don't redo it here.
-                if (closeTyping)
-                    markMisspellings(oldAdjacentWords);
-
-                // This only erases a marker in the first word of the selection.
-                // Perhaps peculiar, but it matches AppKit.
-                document()->removeMarkers(newAdjacentWords.toRange().get(), DocumentMarker::Spelling);
-            }
+            // When typing we check spelling elsewhere, so don't redo it here.
+            if (closeTyping && oldAdjacentWords != newAdjacentWords)
+                markMisspellings(oldAdjacentWords);
+
+            // This only erases a marker in the first word of the selection.
+            // Perhaps peculiar, but it matches AppKit.
+            document()->removeMarkers(newAdjacentWords.toRange().get(), DocumentMarker::Spelling);
         } else
             // When continuous spell checking is off, no markers appear after the selection changes.
             document()->removeMarkers(DocumentMarker::Spelling);