Changeset 89322 in webkit

Timestamp:

Jun 20, 2011 5:57:31 PM (13 years ago)

Author:

ap@apple.com

Message:

2011-06-20 Alexey Proskuryakov <​ap@apple.com>

Reviewed by Darin Adler.

startWordBoundary() in visible_units.cpp assumes that all characters are 16 bit
​https://bugs.webkit.org/show_bug.cgi?id=63027

No new tests, because I don't see this affect any webkit.org ports.

• editing/visible_units.cpp: (WebCore::startWordBoundary): Previous offset is not just minus one - we need to use a proper macro to iterate.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• editing/visible_units.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-06-20  Alexey Proskuryakov  <ap@apple.com>
+
+        Reviewed by Darin Adler.
+
+        startWordBoundary() in visible_units.cpp assumes that all characters are 16 bit
+        https://bugs.webkit.org/show_bug.cgi?id=63027
+
+        No new tests, because I don't see this affect any webkit.org ports.
+
+        * editing/visible_units.cpp: (WebCore::startWordBoundary): Previous offset is not just minus
+        one - we need to use a proper macro to iterate.
+
 2011-06-20  Jay Civelli  <jcivelli@chromium.org>
 

trunk/Source/WebCore/editing/visible_units.cpp

@@ -233 +233 @@
     needMoreContext = false;
     int start, end;
-    findWordBoundary(characters, length, offset - 1, &start, &end);
+    U16_BACK_1(characters, 0, offset);
+    findWordBoundary(characters, length, offset, &start, &end);
     return start;
 }