[SCM] WebKit Debian packaging branch, webkit-1.2, updated. upstream/1.1.90-6072-g9a69373

[enrica at apple.com]

The following commit has been merged in the webkit-1.2 branch:
commit e63d06f1bfcd890f81b44a1202a1c5a0572d3346
Author: enrica at apple.com <enrica at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Dec 3 21:43:38 2009 +0000

    SpinTracer: 349 spins in Mail at WebCore::Editor::advanceToNextMisspelling(bool)
    <rdar://problem/7198592>
    https://bugs.webkit.org/show_bug.cgi?id=32129
    
    Reviewed by John Sullivan.
    
    Fixed the way the next paragraph is calculated.
    I've added a repro case in Safari and attached it to the Bugzilla bug.
    
    * editing/Editor.cpp:
    (WebCore::findFirstMisspellingOrBadGrammarInRange): Changed the way we move to the
    next paragraph in the loop.
    
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@51659 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index ada0ddd..e6d71fc 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,18 @@
+2009-12-03  Enrica Casucci  <enrica at apple.com>
+
+        Reviewed by John Sullivan.
+
+        SpinTracer: 349 spins in Mail at WebCore::Editor::advanceToNextMisspelling(bool)
+        <rdar://problem/7198592>
+        https://bugs.webkit.org/show_bug.cgi?id=32129
+
+        Fixed the way the next paragraph is calculated.
+        I've added a repro case in Safari and attached it to the Bugzilla bug.
+
+        * editing/Editor.cpp:
+        (WebCore::findFirstMisspellingOrBadGrammarInRange): Changed the way we move to the
+        next paragraph in the loop.
+
 2009-12-03  Oliver Hunt  <oliver at apple.com>
 
         Build fix
diff --git a/WebCore/editing/Editor.cpp b/WebCore/editing/Editor.cpp
index 40aed60..12ca958 100644
--- a/WebCore/editing/Editor.cpp
+++ b/WebCore/editing/Editor.cpp
@@ -1813,8 +1813,9 @@ static String findFirstMisspellingOrBadGrammarInRange(EditorClient* client, Rang
         }
         if (lastIteration || totalLengthProcessed + currentLength >= totalRangeLength)
             break;
-        setStart(paragraphRange.get(), startOfNextParagraph(paragraphRange->endPosition()));
-        setEnd(paragraphRange.get(), endOfParagraph(paragraphRange->startPosition()));
+        VisiblePosition newParagraphStart = startOfNextParagraph(paragraphRange->endPosition());
+        setStart(paragraphRange.get(), newParagraphStart);
+        setEnd(paragraphRange.get(), endOfParagraph(newParagraphStart));
         firstIteration = false;
         totalLengthProcessed += currentLength;
     }

-- 
WebKit Debian packaging