Fix some test failures caused by r28019
authoraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Nov 2007 01:53:33 +0000 (01:53 +0000)
committeraroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 26 Nov 2007 01:53:33 +0000 (01:53 +0000)
         Now that stdout is in binary mode, we need to always use printf
         instead of wprintf. Otherwise we'll end up with UTF-16 characters in
         the output.

         Reviewed by Sam.

         * DumpRenderTree/win/UIDelegate.cpp: Replaced uses of wprintf with
         printf.
         (UIDelegate::runJavaScriptAlertPanelWithMessage):
         (UIDelegate::runJavaScriptConfirmPanelWithMessage):
         (UIDelegate::runJavaScriptTextInputPanelWithPrompt):
         (UIDelegate::webViewAddMessageToConsole):

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

WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/win/UIDelegate.cpp

index f2007ec8a7ac9240e5b1100435aa440a0f7389c4..242ad546dc74286c8e45bd7279b25e257b832aac 100644 (file)
@@ -1,3 +1,20 @@
+2007-11-25  Adam Roben  <aroben@apple.com>
+
+        Fix some test failures caused by r28019
+
+        Now that stdout is in binary mode, we need to always use printf
+        instead of wprintf. Otherwise we'll end up with UTF-16 characters in
+        the output.
+
+        Reviewed by Sam.
+
+        * DumpRenderTree/win/UIDelegate.cpp: Replaced uses of wprintf with
+        printf.
+        (UIDelegate::runJavaScriptAlertPanelWithMessage):
+        (UIDelegate::runJavaScriptConfirmPanelWithMessage):
+        (UIDelegate::runJavaScriptTextInputPanelWithPrompt):
+        (UIDelegate::webViewAddMessageToConsole):
+
 2007-11-25  Adam Roben  <aroben@apple.com>
 
         Set the font smoothing preference in DRT
index d101d9697eca2ef106ed43ec732fd0e227d3a9fc..53242954d6287bc930957c4f9b5b772380a6e3a2 100755 (executable)
@@ -107,7 +107,7 @@ HRESULT STDMETHODCALLTYPE UIDelegate::runJavaScriptAlertPanelWithMessage(
         /* [in] */ IWebView* /*sender*/,
         /* [in] */ BSTR message)
 {
-    wprintf(L"ALERT: %s\n", message ? message : L"");
+    printf("ALERT: %S\n", message ? message : L"");
 
     return S_OK;
 }
@@ -117,7 +117,7 @@ HRESULT STDMETHODCALLTYPE UIDelegate::runJavaScriptConfirmPanelWithMessage(
     /* [in] */ BSTR message,
     /* [retval][out] */ BOOL* result)
 {
-    wprintf(L"CONFIRM: %s\n", message ? message : L"");
+    printf("CONFIRM: %S\n", message ? message : L"");
     *result = TRUE;
 
     return S_OK;
@@ -129,7 +129,7 @@ HRESULT STDMETHODCALLTYPE UIDelegate::runJavaScriptTextInputPanelWithPrompt(
     /* [in] */ BSTR defaultText,
     /* [retval][out] */ BSTR *result)
 {
-    wprintf(L"PROMPT: %s, default text: %s\n", message ? message : L"", defaultText ? defaultText : L"");
+    printf("PROMPT: %S, default text: %S\n", message ? message : L"", defaultText ? defaultText : L"");
     *result = SysAllocString(defaultText);
 
     return S_OK;
@@ -142,7 +142,7 @@ HRESULT STDMETHODCALLTYPE UIDelegate::webViewAddMessageToConsole(
     /* [in] */ BSTR url,
     /* [in] */ BOOL isError)
 {
-    wprintf(L"CONSOLE MESSAGE: line %d: %s\n", lineNumber, message ? message : L"");
+    printf("CONSOLE MESSAGE: line %d: %S\n", lineNumber, message ? message : L"");
 
     return S_OK;
 }