- another attempt to fix the Windows build
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 May 2006 06:04:29 +0000 (06:04 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 May 2006 06:04:29 +0000 (06:04 +0000)
        * bindings/js/kjs_window.cpp: (KJS::isSeparator): Take a UChar, not a KJS::UChar.
        * bridge/win/FrameWin.cpp:
        (WebCore::FrameWin::runJavaScriptAlert): Use UChar instead of QChar.
        (WebCore::FrameWin::runJavaScriptConfirm): Ditto.
        * editing/TextIterator.h: Include DeprecatedString.h.

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

WebCore/ChangeLog
WebCore/bindings/js/kjs_window.cpp
WebCore/bridge/win/FrameWin.cpp
WebCore/editing/TextIterator.h

index 96bf0bc3c0b77b860e8ceb3d8f0dc892f4baae28..e158e5c4cee425828f95486064e6542e7d403ea0 100644 (file)
@@ -1,3 +1,13 @@
+2006-05-09  Darin Adler  <darin@apple.com>
+
+        - another attempt to fix the Windows build
+
+        * bindings/js/kjs_window.cpp: (KJS::isSeparator): Take a UChar, not a KJS::UChar.
+        * bridge/win/FrameWin.cpp:
+        (WebCore::FrameWin::runJavaScriptAlert): Use UChar instead of QChar.
+        (WebCore::FrameWin::runJavaScriptConfirm): Ditto.
+        * editing/TextIterator.h: Include DeprecatedString.h.
+
 2006-05-09  Darin Adler  <darin@apple.com>
 
         Reviewed by Hyatt.
index 9a5cd1d1940fe1c1f1b055f14f08aa8acef2a838..763dd8a00c2e8e584b7847b4bf3a485e25e080c4 100644 (file)
@@ -1378,7 +1378,7 @@ static void setWindowFeature(const String& keyString, const String& valueString,
 }
 
 // Though isspace() considers \t and \v to be whitespace, Win IE doesn't.
-static bool isSeparator(UChar c)
+static bool isSeparator(::UChar c)
 {
     return c == ' ' || c == '\t' || c == '\n' || c == '\r' || c == '=' || c == ',' || c == '\0';
 }
index 6bf3aff01facfd671bbd5887807f3f681b21b0dd..47e30b027b54dc136a371c85b48705188e8f9c2c 100644 (file)
@@ -89,19 +89,19 @@ String FrameWin::userAgent() const
 void FrameWin::runJavaScriptAlert(String const& message)
 {
     String text = message;
-    text.replace(QChar('\\'), backslashAsCurrencySymbol());
-    QChar nullChar('\0');
+    text.replace('\\', backslashAsCurrencySymbol());
+    UChar nullChar = 0;
     text += String(&nullChar, 1);
-    MessageBox(view()->windowHandle(), (LPCWSTR)text.unicode(), L"JavaScript Alert", MB_OK);
+    MessageBox(view()->windowHandle(), text.characters(), L"JavaScript Alert", MB_OK);
 }
 
 bool FrameWin::runJavaScriptConfirm(String const& message)
 {
     String text = message;
-    text.replace(QChar('\\'), backslashAsCurrencySymbol());
-    QChar nullChar('\0');
+    text.replace('\\', backslashAsCurrencySymbol());
+    UChar nullChar = 0;
     text += String(&nullChar, 1);
-    return (MessageBox(view()->windowHandle(), (LPCWSTR)text.unicode(), L"JavaScript Alert", MB_OKCANCEL) == IDOK);
+    return MessageBox(view()->windowHandle(), text.characters(), L"JavaScript Alert", MB_OKCANCEL) == IDOK;
 }
 
 // FIXME: This needs to be unified with the keyPress method on FrameMac
index eb23553979eaa46c0ca7844e6fc86c428f79fcd4..0564605c03ef1d5bcedd34060a5fee2ff9bf492d 100644 (file)
@@ -26,6 +26,7 @@
 #ifndef KHTML_EDITING_VISIBLE_TEXT_H
 #define KHTML_EDITING_VISIBLE_TEXT_H
 
+#include "DeprecatedString.h"
 #include "InlineTextBox.h"
 #include "Range.h"
 #include <wtf/Vector.h>