Reviewed by Sam.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 15 Oct 2007 18:47:56 +0000 (18:47 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 15 Oct 2007 18:47:56 +0000 (18:47 +0000)
        - added logging of window.prompt and window.confirm

        * DumpRenderTree/mac/UIDelegate.mm:
        (-[UIDelegate webView:runJavaScriptAlertPanelWithMessage:initiatedByFrame:]):
        Implement the recommended delegate rather than the deprecated one.
        (-[UIDelegate webView:runJavaScriptConfirmPanelWithMessage:initiatedByFrame:]):
        Added.
        (-[UIDelegate webView:runJavaScriptTextInputPanelWithPrompt:defaultText:initiatedByFrame:]):
        Added.

        * DumpRenderTree/win/UIDelegate.h:
        * DumpRenderTree/win/UIDelegate.cpp:
        (UIDelegate::runJavaScriptAlertPanelWithMessage): Got rid of unnecessary check for null string.
        (UIDelegate::runJavaScriptConfirmPanelWithMessage): Added.
        (UIDelegate::runJavaScriptTextInputPanelWithPrompt): Added.
        (UIDelegate::webViewAddMessageToConsole): Got rid of unnecessary check for null string.

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

WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/mac/UIDelegate.mm
WebKitTools/DumpRenderTree/win/UIDelegate.cpp
WebKitTools/DumpRenderTree/win/UIDelegate.h

index a115321c2e67c693c0785cc4d553030a3af9782a..d04887cc02e7431f828d8d6d103a2cef715c7fe6 100644 (file)
@@ -1,3 +1,24 @@
+2007-10-15  Darin Adler  <darin@apple.com>
+
+        Reviewed by Sam.
+
+        - added logging of window.prompt and window.confirm
+
+        * DumpRenderTree/mac/UIDelegate.mm:
+        (-[UIDelegate webView:runJavaScriptAlertPanelWithMessage:initiatedByFrame:]):
+        Implement the recommended delegate rather than the deprecated one.
+        (-[UIDelegate webView:runJavaScriptConfirmPanelWithMessage:initiatedByFrame:]):
+        Added.
+        (-[UIDelegate webView:runJavaScriptTextInputPanelWithPrompt:defaultText:initiatedByFrame:]):
+        Added.
+
+        * DumpRenderTree/win/UIDelegate.h:
+        * DumpRenderTree/win/UIDelegate.cpp:
+        (UIDelegate::runJavaScriptAlertPanelWithMessage): Got rid of unnecessary check for null string.
+        (UIDelegate::runJavaScriptConfirmPanelWithMessage): Added.
+        (UIDelegate::runJavaScriptTextInputPanelWithPrompt): Added.
+        (UIDelegate::webViewAddMessageToConsole): Got rid of unnecessary check for null string.
+
 2007-10-15  Alice Liu  <alice.liu@apple.com>
 
         Reviewed by Sam Weinig.
index a8d103601e96e304c58d46ce728fa62fa75683d7..dc80744cb6e80561d1a309031e5d995ba32daa21 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2006 Apple Computer, Inc.  All rights reserved.
+ * Copyright (C) 2006. 2007 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -59,12 +59,26 @@ DumpRenderTreeDraggingInfo *draggingInfo = nil;
     printf ("CONSOLE MESSAGE: line %d: %s\n", [lineNumber intValue], [message UTF8String]);
 }
 
-- (void)webView:(WebView *)sender runJavaScriptAlertPanelWithMessage:(NSString *)message
+- (void)webView:(WebView *)sender runJavaScriptAlertPanelWithMessage:(NSString *)message initiatedByFrame:(WebFrame *)frame
 {
     if (!done)
         printf("ALERT: %s\n", [message UTF8String]);
 }
 
+- (BOOL)webView:(WebView *)sender runJavaScriptConfirmPanelWithMessage:(NSString *)message initiatedByFrame:(WebFrame *)frame
+{
+    if (!done)
+        printf("CONFIRM: %s\n", [message UTF8String]);
+    return YES;
+}
+
+- (NSString *)webView:(WebView *)sender runJavaScriptTextInputPanelWithPrompt:(NSString *)prompt defaultText:(NSString *)defaultText initiatedByFrame:(WebFrame *)frame
+{
+    if (!done)
+        printf("PROMPT: %s, default text: %s\n", [prompt UTF8String], [defaultText UTF8String]);
+    return defaultText;
+}
+
 - (void)webView:(WebView *)sender dragImage:(NSImage *)anImage at:(NSPoint)viewLocation offset:(NSSize)initialOffset event:(NSEvent *)event pasteboard:(NSPasteboard *)pboard source:(id)sourceObj slideBack:(BOOL)slideFlag forView:(NSView *)view
 {
      assert(!draggingInfo);
index f9162147f8753317a34e79b7c1bb1ac724fb312f..39a4abbb8bc322c3266af77e0b705f3af9ae76de 100755 (executable)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2005, 2006, 2007 Apple Inc.  All rights reserved.
+ * Copyright (C) 2005, 2006, 2007 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -56,12 +56,12 @@ HRESULT STDMETHODCALLTYPE UIDelegate::QueryInterface(REFIID riid, void** ppvObje
     return S_OK;
 }
 
-ULONG STDMETHODCALLTYPE UIDelegate::AddRef(void)
+ULONG STDMETHODCALLTYPE UIDelegate::AddRef()
 {
     return ++m_refCount;
 }
 
-ULONG STDMETHODCALLTYPE UIDelegate::Release(void)
+ULONG STDMETHODCALLTYPE UIDelegate::Release()
 {
     ULONG newRef = --m_refCount;
     if (!newRef)
@@ -98,7 +98,30 @@ HRESULT STDMETHODCALLTYPE UIDelegate::runJavaScriptAlertPanelWithMessage(
         /* [in] */ IWebView* /*sender*/,
         /* [in] */ BSTR message)
 {
-    wprintf(L"ALERT: %s\n", message ? message : L"");
+    wprintf(L"ALERT: %s\n", message);
+
+    return S_OK;
+}
+
+HRESULT STDMETHODCALLTYPE UIDelegate::runJavaScriptConfirmPanelWithMessage( 
+    /* [in] */ IWebView* sender,
+    /* [in] */ BSTR message,
+    /* [retval][out] */ BOOL* result)
+{
+    wprintf(L"CONFIRM: %s\n", message);
+    *result = TRUE;
+
+    return S_OK;
+}
+
+HRESULT STDMETHODCALLTYPE UIDelegate::runJavaScriptTextInputPanelWithPrompt( 
+    /* [in] */ IWebView *sender,
+    /* [in] */ BSTR message,
+    /* [in] */ BSTR defaultText,
+    /* [retval][out] */ BSTR *result)
+{
+    wprintf(L"PROMPT: %s, default text: %s\n", message, defaultText);
+    *result = SysAllocString(defaultText);
 
     return S_OK;
 }
@@ -110,7 +133,7 @@ HRESULT STDMETHODCALLTYPE UIDelegate::webViewAddMessageToConsole(
     /* [in] */ BSTR url,
     /* [in] */ BOOL isError)
 {
-    wprintf(L"CONSOLE MESSAGE: line %d: %s\n", lineNumber, message ? message : L"");
+    wprintf(L"CONSOLE MESSAGE: line %d: %s\n", lineNumber, message);
 
     return S_OK;
 }
index ab637c60bfb4f2d4105b5f343ab7d8b43238ca9a..671c648dafc275a0cd42501f8b720e915d1a1f91 100755 (executable)
@@ -124,13 +124,13 @@ public:
     virtual HRESULT STDMETHODCALLTYPE runJavaScriptConfirmPanelWithMessage( 
         /* [in] */ IWebView *sender,
         /* [in] */ BSTR message,
-        /* [retval][out] */ BOOL *result) { return E_NOTIMPL; }
+        /* [retval][out] */ BOOL *result);
 
     virtual HRESULT STDMETHODCALLTYPE runJavaScriptTextInputPanelWithPrompt( 
         /* [in] */ IWebView *sender,
         /* [in] */ BSTR message,
         /* [in] */ BSTR defaultText,
-        /* [retval][out] */ BSTR *result) { return E_NOTIMPL; }
+        /* [retval][out] */ BSTR *result);
 
     virtual HRESULT STDMETHODCALLTYPE runBeforeUnloadConfirmPanelWithMessage( 
         /* [in] */ IWebView *sender,