WebCore:
authorrwlbuis <rwlbuis@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Oct 2006 12:44:16 +0000 (12:44 +0000)
committerrwlbuis <rwlbuis@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Oct 2006 12:44:16 +0000 (12:44 +0000)
        Reviewed by Maciej.

        Specialization of alert() for DumpRenderTree - just log the output, don't show any message box.

WebKitTools:
        Reviewed by Maciej.

        Specialization of alert() for DumpRenderTree - just log the output, don't show any message box.

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

WebCore/ChangeLog
WebCore/platform/qt/FrameQt.cpp
WebCore/platform/qt/FrameQtClient.cpp
WebCore/platform/qt/FrameQtClient.h
WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/DumpRenderTree.qtproj/DumpRenderTreeClient.cpp
WebKitTools/DumpRenderTree/DumpRenderTree.qtproj/DumpRenderTreeClient.h

index cbfe6a5e477ce469051ac62d1c777ccc3d52ceea..e22215905aa36f618fc0cdd3ad1a42c1f8e31a61 100644 (file)
@@ -1,3 +1,15 @@
+2006-10-12  Nikolas Zimmermann  <zimmermann@kde.org>
+
+        Reviewed by Maciej.
+
+        Specialization of alert() for DumpRenderTree - just log the output, don't show any message box.
+
+        * platform/qt/FrameQt.cpp:
+        (WebCore::FrameQt::runJavaScriptAlert): Move runJavaScriptAlert functionality to FrameQtClientDefault.
+        * platform/qt/FrameQtClient.cpp:
+        (WebCore::FrameQtClientDefault::runJavaScriptAlert):
+        * platform/qt/FrameQtClient.h:
+
 2006-10-12  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Eric.
index df89fab9eeff565ffd7d6461646d6bceee054e44..36424b31509d0317ab0d8cb73babf9ad1dbefc25 100644 (file)
@@ -58,7 +58,6 @@
 #include "TypingCommand.h"
 
 #include <QScrollArea>
-#include <QMessageBox>
 
 #define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
 
@@ -157,7 +156,7 @@ String FrameQt::userAgent() const
 
 void FrameQt::runJavaScriptAlert(String const& message)
 {
-    QMessageBox::information(view()->qwidget(), "JavaScript", message);
+    m_client->runJavaScriptAlert(message);
 }
 
 bool FrameQt::runJavaScriptConfirm(String const& message)
index 1f6f0527de47b1dd88ab70f90df01b4a959f4880..00c6eb0e3574f753b2a2f16c4d52a14782112bf2 100644 (file)
@@ -32,6 +32,8 @@
 #include "ResourceLoader.h"
 #include "ResourceLoaderInternal.h"
 
+#include <QMessageBox>
+
 namespace WebCore {
 
 FrameQtClientDefault::FrameQtClientDefault()
@@ -69,6 +71,11 @@ void FrameQtClientDefault::submitForm(const String& method, const KURL& url, con
     loader->start(0);
 }
 
+void FrameQtClientDefault::runJavaScriptAlert(String const& message)
+{
+    QMessageBox::information(m_frame->view()->qwidget(), "JavaScript", message);
+}
+
 void FrameQtClientDefault::receivedResponse(ResourceLoader*, PlatformResponse)
 {
     // no-op
index fc81b5472c08fbba20c1aac24e846c8633d317fd..cbb3b562a5cb5088599a9fce7c8ba9d38de34e33 100644 (file)
@@ -44,6 +44,9 @@ public:
 
     virtual void openURL(const KURL&) = 0;
     virtual void submitForm(const String& method, const KURL&, const FormData*) = 0;
+
+    // WebKitPart / DumpRenderTree want to handle this differently.
+    virtual void runJavaScriptAlert(String const& message) = 0;
 };
 
 class FrameQtClientDefault : public FrameQtClient,
@@ -59,6 +62,8 @@ public:
     virtual void openURL(const KURL&);
     virtual void submitForm(const String& method, const KURL&, const FormData*);
 
+    virtual void runJavaScriptAlert(String const& message);
+
     // ResourceLoaderClient
     virtual void receivedResponse(ResourceLoader*, PlatformResponse);
     virtual void receivedData(ResourceLoader*, const char*, int);
index 47446a8d4609a85e94d765d4222f21c2213c0ab0..5aa290cb78b6d493ab5ef3e68ea92e55fb624de2 100644 (file)
@@ -1,3 +1,13 @@
+2006-10-12  Nikolas Zimmermann  <zimmermann@kde.org>
+
+        Reviewed by Maciej.
+
+        Specialization of alert() for DumpRenderTree - just log the output, don't show any message box.
+
+        * DumpRenderTree/DumpRenderTree.qtproj/DumpRenderTreeClient.cpp:
+        (WebCore::DumpRenderTreeClient::runJavaScriptAlert):
+        * DumpRenderTree/DumpRenderTree.qtproj/DumpRenderTreeClient.h:
+
 2006-10-12  Krzysztof Kowalczyk  <kkowalczyk@gmail.com>
 
         Reviewed by Maciej.
index 9310b621b8b3dd5217a0558ef1e94282bbc50799..b1dca2a696f8281d1cdb7fa3dc0d5e889661901c 100644 (file)
@@ -30,6 +30,8 @@
 
 #include "DumpRenderTree.h"
 
+#include <QDebug>
+
 namespace WebCore {
 
 DumpRenderTreeClient::DumpRenderTreeClient()
@@ -41,6 +43,11 @@ DumpRenderTreeClient::~DumpRenderTreeClient()
 {
 }
 
+void DumpRenderTreeClient::runJavaScriptAlert(String const& message)
+{
+    qDebug() << "ALERT: " << message << "\n";
+}
+
 }
 
 // vim: ts=4 sw=4 et
index 923a6d929def1abd230295d17d0b53805bd28036..f780b9ff978dfc8d337bc200c5466d969d9aa435 100644 (file)
@@ -37,6 +37,8 @@ class DumpRenderTreeClient : public FrameQtClientDefault
 public:
     DumpRenderTreeClient();
     virtual ~DumpRenderTreeClient();
+
+    virtual void runJavaScriptAlert(String const& message);
 };
 
 }