2006-11-03 Zack Rusin <zack@kde.org>
authorstaikos <staikos@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 3 Nov 2006 10:48:15 +0000 (10:48 +0000)
committerstaikos <staikos@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 3 Nov 2006 10:48:15 +0000 (10:48 +0000)
        Reviewed by Maciej, landed by George.

Compile on both KDE and Qt platforms

* CMakeLists.txt: add zlib explicitly
* platform/network/qt/ResourceHandleManager.cpp:
(WebCore::ResourceHandleManager::self):
(WebCore::QtJob::QtJob):
(WebCore::QtJob::timerEvent): Integrate both implementations
      so that moc can parse it
(WebCore::ResourceHandleManager::deliverJobData):
(WebCore::ResourceHandleManager::remove):
* platform/network/qt/ResourceHandleManager.h:

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

WebCore/CMakeLists.txt
WebCore/ChangeLog
WebCore/platform/network/qt/ResourceHandleManager.cpp
WebCore/platform/network/qt/ResourceHandleManager.h
WebKitQt/ChangeLog
WebKitQt/QtLauncher/main.cpp
WebKitQt/WebKitPart/WebKitPartBrowserExtension.cpp

index 0186a7894ddda484890b6e5bdb0663f6c712f8cf..885e5a2b8a775f7f6c466095353b9f6087f83af4 100644 (file)
@@ -1207,6 +1207,7 @@ target_link_libraries(WebCore-unity
   wtf-unity
   kjs-unity
   pcre-unity
+  z
 )
 ELSE (WEBKIT_USE_KDE_SUPPORT)
 target_link_libraries(WebCore-unity
@@ -1218,6 +1219,7 @@ target_link_libraries(WebCore-unity
   wtf-unity
   kjs-unity
   pcre-unity
+  z
 )
 ENDIF (WEBKIT_USE_KDE_SUPPORT)
 
index a3c2b5cf6407a8a45c4160ffa21a2e64c3783610..2c3f3274d673741aad09cfaf129de6ac16186566 100644 (file)
@@ -1,3 +1,19 @@
+2006-11-03  Zack Rusin  <zack@kde.org>
+
+        Reviewed by Maciej.
+
+        Compile on both KDE and Qt platforms
+
+        * CMakeLists.txt: add zlib explicitly
+        * platform/network/qt/ResourceHandleManager.cpp:
+        (WebCore::ResourceHandleManager::self):
+        (WebCore::QtJob::QtJob):
+        (WebCore::QtJob::timerEvent): Integrate both implementations
+        so that moc can parse it
+        (WebCore::ResourceHandleManager::deliverJobData):
+        (WebCore::ResourceHandleManager::remove):
+        * platform/network/qt/ResourceHandleManager.h:
+
 2006-11-03  Maciej Stachowiak  <mjs@apple.com>
 
         No review, suggested by Mitz.
index 3bf029c6129915239c2896be83e399dca7a87abf..57a5b144282c9a45df0796fee6d138f308c210cb 100644 (file)
@@ -22,7 +22,7 @@
  * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
@@ -42,6 +42,36 @@ namespace WebCore {
 
 static ResourceHandleManager* s_self = 0;
 
+
+ResourceHandleManager* ResourceHandleManager::self()
+{
+    if (!s_self)
+        s_self = new ResourceHandleManager();
+
+    return s_self;
+}
+
+QtJob::QtJob(const QString& path)
+    : m_path(path)
+{
+    startTimer(0);
+}
+
+void QtJob::timerEvent(QTimerEvent* e)
+{
+    killTimer(e->timerId());
+
+    QFile f(m_path);
+    QByteArray data;
+    if (f.open(QIODevice::ReadOnly)) {
+        data = f.readAll();
+        f.close();
+    };
+
+    emit finished(this, data);
+
+    deleteLater();
+}
 #if PLATFORM(KDE)
 
 ResourceHandleManager::ResourceHandleManager()
@@ -55,14 +85,11 @@ ResourceHandleManager::~ResourceHandleManager()
 {
 }
 
-ResourceHandleManager* ResourceHandleManager::self()
+
+void ResourceHandleManager::deliverJobData(QtJob*, const QByteArray&)
 {
-    if (!s_self)
-        s_self = new ResourceHandleManager();
 
-    return s_self;
 }
-
 void ResourceHandleManager::slotData(KIO::Job* kioJob, const QByteArray& data)
 {
     ResourceHandle* job = 0;
@@ -148,7 +175,7 @@ void ResourceHandleManager::remove(ResourceHandle* job)
         // This must be called before didFinishLoading(),
         // otherwhise assembleResponseHeaders() is called too early...
         RefPtr<PlatformResponseQt> response(new PlatformResponseQt());
-        response->data = headers;    
+        response->data = headers;
         response->url = job->url().url();
 
         job->receivedResponse(response);
@@ -199,31 +226,8 @@ void ResourceHandleManager::cancel(ResourceHandle* job)
     job->setError(1);
 }
 
-#else
 // Qt Resource Handle Manager
-
-QtJob::QtJob(const QString& path)
-    : m_path(path)
-{
-    startTimer(0);
-}
-
-void QtJob::timerEvent(QTimerEvent* e)
-{
-    killTimer(e->timerId());
-
-    QFile f(m_path);
-    QByteArray data;
-    if (f.open(QIODevice::ReadOnly)) {
-        data = f.readAll();
-        f.close();
-    };
-
-    emit finished(this, data);
-
-    deleteLater();
-}
-
+#else
 ResourceHandleManager::ResourceHandleManager()
     : m_frameClient(0)
 {
@@ -233,13 +237,6 @@ ResourceHandleManager::~ResourceHandleManager()
 {
 }
 
-ResourceHandleManager* ResourceHandleManager::self()
-{
-    if (!s_self)
-        s_self = new ResourceHandleManager();
-
-    return s_self;
-}
 
 void ResourceHandleManager::remove(ResourceHandle* job)
 {
index ab8258c69890b3e0ecea6ce385c23a155f646108..8780701e85eaa45603f8c11ecfa5abbeb14c04a3 100644 (file)
@@ -37,34 +37,7 @@ namespace WebCore {
 
 class FrameQtClient;
 
-#if PLATFORM(KDE)
-class ResourceHandleManager : public QObject {
-Q_OBJECT
-public:
-    static ResourceHandleManager* self();
-
-    void add(ResourceHandle*, FrameQtClient*);
-    void cancel(ResourceHandle*);
-
-public Q_SLOTS:
-    void slotData(KIO::Job*, const QByteArray& data);
-    void slotMimetype(KIO::Job*, const QString& type);
-    void slotResult(KJob*);
     
-private:
-    ResourceHandleManager();
-    ~ResourceHandleManager();
-
-    void remove(ResourceHandle*);
-
-    // KIO Job <-> WebKit Job mapping
-    QMap<ResourceHandle*, KIO::Job*> m_jobToKioMap;
-    QMap<KIO::Job*, ResourceHandle*> m_kioToJobMap;
-
-    FrameQtClient* m_frameClient;
-};
-#else
-
 class QtJob : public QObject
 {
     Q_OBJECT
@@ -81,8 +54,9 @@ private:
     QString m_path;
 };
 
-class ResourceHandleManager : public QObject {
-Q_OBJECT
+class ResourceHandleManager : public QObject
+{
+    Q_OBJECT
 public:
     static ResourceHandleManager* self();
 
@@ -90,22 +64,30 @@ public:
     void cancel(ResourceHandle*);
 
 public Q_SLOTS:
-    void deliverJobData(QtJob* job, const QByteArray& data);
-
+    void slotData(KIO::Job*, const QByteArray& data);
+    void slotMimetype(KIO::Job*, const QString& type);
+    void slotResult(KJob*);
+    void deliverJobData(QtJob* , const QByteArray&);
+    
 private:
     ResourceHandleManager();
     ~ResourceHandleManager();
 
     void remove(ResourceHandle*);
 
+#if PLATFORM(KDE)
+    // KIO Job <-> WebKit Job mapping
+    QMap<ResourceHandle*, KIO::Job*> m_jobToKioMap;
+    QMap<KIO::Job*, ResourceHandle*> m_kioToJobMap;
+#else
+    
     QMap<ResourceHandle*, QtJob*> m_resourceToJob;
     QMap<QtJob*, ResourceHandle*> m_jobToResource;
+#endif
 
     FrameQtClient* m_frameClient;
 };
 
-#endif
-
 }
 
 #endif
index 74b1e10debd3afc59bb1662793b1cc81b7e7bde8..ee6616e1e7d8e5632b3153b2a90205532f201fd7 100644 (file)
@@ -1,3 +1,14 @@
+2006-11-03  Zack Rusin  <zack@kde.org>
+
+        Reviewed by Maciej.
+
+        Fixing compile on both Qt and KDE platforms.
+
+        * QtLauncher/main.cpp:
+        (main): url has already been defined
+        * WebKitPart/WebKitPartBrowserExtension.cpp: removing
+        duplicate implementation of this class
+
 2006-10-31  Simon Hausmann  <hausmann@kde.org>
 
         Reviewed by Maciej, landed by Anders.
index 534e92dd18551dc7f5309bd76ee0120688a0646c..adaa014991b473fd4ad913400c55a8e6a983a74a 100644 (file)
@@ -65,18 +65,14 @@ int main(int argc, char **argv)
     KApplication app;
     KCmdLineArgs *args = KCmdLineArgs::parsedArgs();
 
-    QString url;
-
     if (args->count() != 0)
         url = args->arg(0);
-
 #else
     QApplication app(argc, argv);
 
     const QStringList args = app.arguments();
     if (args.count() > 1)
         url = args.at(1);
-
 #endif
      
     QWidget topLevel;
index 149b85d5c8acf5139617ba5d2fedbb8f8d6bbc2b..66bf8542cf021a0811d77dff7d61867886637c36 100644 (file)
@@ -38,43 +38,3 @@ WebKitPartBrowserExtension::~WebKitPartBrowserExtension()
 }
 
 // vim: ts=4 sw=4 et
-/*
- * Copyright (C) 2006 Nikolas Zimmermann <zimmermann@kde.org>
- * 
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE COMPUTER, INC. ``AS IS'' AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL APPLE COMPUTER, INC. OR
- * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
- * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
- * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
- * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
- * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "WebKitPartBrowserExtension.h"
-
-WebKitPartBrowserExtension::WebKitPartBrowserExtension(KParts::ReadOnlyPart* part)
-    : KParts::BrowserExtension(part)
-{
-}
-
-WebKitPartBrowserExtension::~WebKitPartBrowserExtension()
-{
-}
-
-// vim: ts=4 sw=4 et