Changeset 79830 in webkit

Timestamp:

Feb 27, 2011 1:51:08 PM (13 years ago)

Author:

Adam Roben

Message:

Use iswalpha instead of isalpha when dealing with wchar_ts in EditingDelegate

Fixes <​http://webkit.org/b/55062> <rdar://problem/9059907> Crash beneath
EditingDelegate::checkSpellingOfString when running fast/forms/input-text-maxlength.html or
fast/forms/input-text-paste-maxlength.html on Windows with full page heap enabled

Reviewed by Anders Carlsson.

• DumpRenderTree/win/EditingDelegate.cpp:

(indexOfFirstWordCharacter):
(wordLength):
Use iswalpha instead of isalpha, since TCHAR is really just wchar_t.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/win/EditingDelegate.cpp (modified) (2 diffs)

trunk/Tools/ChangeLog

@@ +1 @@
+2011-02-27  Adam Roben  <aroben@apple.com>
+
+        Use iswalpha instead of isalpha when dealing with wchar_ts in EditingDelegate
+
+        Fixes <http://webkit.org/b/55062> <rdar://problem/9059907> Crash beneath
+        EditingDelegate::checkSpellingOfString when running fast/forms/input-text-maxlength.html or
+        fast/forms/input-text-paste-maxlength.html on Windows with full page heap enabled
+
+        Reviewed by Anders Carlsson.
+
+        * DumpRenderTree/win/EditingDelegate.cpp:
+        (indexOfFirstWordCharacter):
+        (wordLength):
+        Use iswalpha instead of isalpha, since TCHAR is really just wchar_t.
+
 2011-02-27  Adam Roben  <aroben@apple.com>
 

trunk/Tools/DumpRenderTree/win/EditingDelegate.cpp

@@ -358 +358 @@
 {
     const TCHAR* cursor = text;
-    while (*cursor && !isalpha(*cursor))
+    while (*cursor && !iswalpha(*cursor))
         ++cursor;
     return *cursor ? (cursor - text) : -1;
@@ -366 +366 @@
 {
     const TCHAR* cursor = text;
-    while (*cursor && isalpha(*cursor))
+    while (*cursor && iswalpha(*cursor))
         ++cursor;
     return cursor - text;