Check for null BSTR that can be passed to the UIDelegate methods
authorweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Oct 2007 03:47:16 +0000 (03:47 +0000)
committerweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Oct 2007 03:47:16 +0000 (03:47 +0000)
        from javascript null and undefined.  Fixes a failing test case on
        Windows (fast/dom/Window/alert-undefined.html)

        Reviewed by Eric Seidel.

        * DumpRenderTree/win/UIDelegate.cpp:
        (UIDelegate::runJavaScriptAlertPanelWithMessage):
        (UIDelegate::runJavaScriptConfirmPanelWithMessage):
        (UIDelegate::runJavaScriptTextInputPanelWithPrompt):
        (UIDelegate::webViewAddMessageToConsole):

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

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

index b6dc2637c31697c3532b4123b292023e62dcdd16..81abb96938d6f2d06ef6a3bf4e3875abcb82beab 100644 (file)
@@ -1,3 +1,17 @@
+2007-10-23  Sam Weinig  <sam@webkit.org>
+
+        Check for null BSTR that can be passed to the UIDelegate methods
+        from javascript null and undefined.  Fixes a failing test case on
+        Windows (fast/dom/Window/alert-undefined.html)
+
+        Reviewed by Eric Seidel.
+
+        * DumpRenderTree/win/UIDelegate.cpp:
+        (UIDelegate::runJavaScriptAlertPanelWithMessage):
+        (UIDelegate::runJavaScriptConfirmPanelWithMessage):
+        (UIDelegate::runJavaScriptTextInputPanelWithPrompt):
+        (UIDelegate::webViewAddMessageToConsole):
+
 2007-10-23  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Darin.
index 39a4abbb8bc322c3266af77e0b705f3af9ae76de..70fc3e812225f0de9b2e7b5c63e094dd04d48a70 100755 (executable)
@@ -98,7 +98,7 @@ HRESULT STDMETHODCALLTYPE UIDelegate::runJavaScriptAlertPanelWithMessage(
         /* [in] */ IWebView* /*sender*/,
         /* [in] */ BSTR message)
 {
-    wprintf(L"ALERT: %s\n", message);
+    wprintf(L"ALERT: %s\n", message ? message : L"");
 
     return S_OK;
 }
@@ -108,7 +108,7 @@ HRESULT STDMETHODCALLTYPE UIDelegate::runJavaScriptConfirmPanelWithMessage(
     /* [in] */ BSTR message,
     /* [retval][out] */ BOOL* result)
 {
-    wprintf(L"CONFIRM: %s\n", message);
+    wprintf(L"CONFIRM: %s\n", message ? message : L"");
     *result = TRUE;
 
     return S_OK;
@@ -120,7 +120,7 @@ HRESULT STDMETHODCALLTYPE UIDelegate::runJavaScriptTextInputPanelWithPrompt(
     /* [in] */ BSTR defaultText,
     /* [retval][out] */ BSTR *result)
 {
-    wprintf(L"PROMPT: %s, default text: %s\n", message, defaultText);
+    wprintf(L"PROMPT: %s, default text: %s\n", message ? message : L"", defaultText ? defaultText : L"");
     *result = SysAllocString(defaultText);
 
     return S_OK;
@@ -133,7 +133,7 @@ HRESULT STDMETHODCALLTYPE UIDelegate::webViewAddMessageToConsole(
     /* [in] */ BSTR url,
     /* [in] */ BOOL isError)
 {
-    wprintf(L"CONSOLE MESSAGE: line %d: %s\n", lineNumber, message);
+    wprintf(L"CONSOLE MESSAGE: line %d: %s\n", lineNumber, message ? message : L"");
 
     return S_OK;
 }