Get rid of FrameQtClient. It's unused since we moved over
authorlars <lars@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Jan 2007 11:16:08 +0000 (11:16 +0000)
committerlars <lars@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Jan 2007 11:16:08 +0000 (11:16 +0000)
        to the loader.

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

12 files changed:
WebCore/ChangeLog
WebCore/WebCore.pro
WebCore/page/qt/FrameQt.cpp
WebCore/page/qt/FrameQt.h
WebCore/page/qt/FrameQtClient.cpp [deleted file]
WebCore/page/qt/FrameQtClient.h [deleted file]
WebCore/platform/network/ResourceHandleInternal.h
WebCore/platform/network/qt/ResourceHandleManagerQt.cpp
WebCore/platform/network/qt/ResourceHandleManagerQt.h
WebCore/platform/network/qt/ResourceHandleQt.cpp
WebKitQt/Api/qwebframe.cpp
WebKitQt/ChangeLog

index 5a32903..02194bc 100644 (file)
@@ -1,3 +1,24 @@
+2007-01-26  Lars Knoll <lars@trolltech.com>
+
+        Reviewed by Zack
+
+        Get rid of FrameQtClient. It's unused since we moved over
+        to the loader.
+
+        * WebCore.pro:
+        * page/qt/FrameQt.cpp:
+        (WebCore::FrameQt::FrameQt):
+        * page/qt/FrameQt.h:
+        * page/qt/FrameQtClient.cpp: Removed.
+        * page/qt/FrameQtClient.h: Removed.
+        * platform/network/ResourceHandleInternal.h:
+        * platform/network/qt/ResourceHandleManagerQt.cpp:
+        (WebCore::RequestQt::RequestQt):
+        (WebCore::ResourceHandleManager::add):
+        * platform/network/qt/ResourceHandleManagerQt.h:
+        * platform/network/qt/ResourceHandleQt.cpp:
+        (WebCore::ResourceHandle::start):
+
 2007-01-26  Zack Rusin  <zack@kde.org>
 
         Fix the compile
index c6a0821..c35c53f 100644 (file)
@@ -472,7 +472,6 @@ SOURCES += \
     page/DragController.cpp \
     page/qt/EventHandlerQt.cpp \
     page/qt/FrameQt.cpp \
-    page/qt/FrameQtClient.cpp \
     page/qt/DragControllerQt.cpp \
     xml/XPathUtil.cpp \
     xml/XPathPredicate.cpp \
index 680927c..7ac29c9 100644 (file)
@@ -98,13 +98,10 @@ static void doScroll(const RenderObject* r, bool isHorizontal, int multiplier)
 }
 #endif
 
-FrameQt::FrameQt(Page* page, HTMLFrameOwnerElement* ownerElement,
-                 FrameQtClient* frameClient, FrameLoaderClient* frameLoader)
+FrameQt::FrameQt(Page* page, HTMLFrameOwnerElement* ownerElement, FrameLoaderClient* frameLoader)
     : Frame(page, ownerElement, frameLoader)
     , m_bindingRoot(0)
 {
-    m_client = frameClient;
-    m_client->setFrame(this);
 }
 
 FrameQt::~FrameQt()
@@ -259,11 +256,6 @@ void FrameQt::setFrameGeometry(const IntRect& r)
     setFrameGeometry(QRect(r));
 }
 
-FrameQtClient* FrameQt::client() const
-{
-    return m_client;
-}
-
 void FrameQt::createNewWindow(const FrameLoadRequest& request, const WindowFeatures& args, Frame*& frame)
 {
     notImplemented();
index b5e7f21..c3e8248 100644 (file)
@@ -34,7 +34,6 @@
 #define FrameQt_h
 
 #include "Frame.h"
-#include "FrameQtClient.h"
 #include "WindowFeatures.h"
 
 class QWidget;
@@ -53,7 +52,7 @@ class EditorClient;
 
 class FrameQt : public Frame {
 public:
-    FrameQt(Page*, HTMLFrameOwnerElement*, FrameQtClient*, FrameLoaderClient*);
+    FrameQt(Page*, HTMLFrameOwnerElement*, FrameLoaderClient*);
     virtual ~FrameQt();
 
     virtual KJS::Bindings::Instance* getEmbedInstanceForWidget(Widget*);
@@ -91,8 +90,6 @@ public:
 
     void setFrameGeometry(const IntRect&);
 
-    FrameQtClient* client() const;
-
     void createNewWindow(const FrameLoadRequest&, const WindowFeatures&, Frame*&);
     void goBackOrForward(int);
 
@@ -103,7 +100,6 @@ private:
     virtual bool isLoadTypeReload();
     virtual bool passMouseDownEventToWidget(Widget*);
 
-    FrameQtClient* m_client;
     bool m_beginCalled : 1;    
 
     KJS::Bindings::RootObject* m_bindingRoot;  // The root object used for objects
diff --git a/WebCore/page/qt/FrameQtClient.cpp b/WebCore/page/qt/FrameQtClient.cpp
deleted file mode 100644 (file)
index f18b788..0000000
+++ /dev/null
@@ -1,151 +0,0 @@
-/*
- * Copyright (C) 2006 Nikolas Zimmermann <zimmermann@kde.org>
- * Copyright (C) 2006 Lars Knoll <lars@trolltech.com>
- *
- * 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 "FrameQtClient.h"
-
-#include "Cache.h"
-#include "FrameQt.h"
-#include "Document.h"
-#include "FrameTree.h"
-#include "FrameView.h"
-#include "FrameLoader.h"
-#include "ResourceHandle.h"
-#include "ResourceHandleInternal.h"
-
-#if PLATFORM(KDE)
-#include <kstdguiitem.h>
-#include <kmessagebox.h>
-#include <kinputdialog.h>
-#else
-#include <QInputDialog>
-#include <QMessageBox>
-#endif
-
-#include <qdebug.h>
-
-namespace WebCore {
-
-FrameQtClient::FrameQtClient()
-    : ResourceHandleClient()
-    , m_frame(0)
-{
-}
-
-FrameQtClient::~FrameQtClient()
-{
-}
-
-void FrameQtClient::setFrame(const FrameQt* frame)
-{
-    ASSERT(frame);
-    m_frame = const_cast<FrameQt*>(frame);
-}
-
-void FrameQtClient::openURL(const KURL& url)
-{
-    ASSERT(m_frame);
-
-    m_frame->loader()->load(url);
-}
-
-void FrameQtClient::submitForm(const String& method, const KURL& url, PassRefPtr<FormData> postData)
-{
-    ASSERT(m_frame);
-
-    ResourceRequest request(url);
-    request.setHTTPMethod(method);
-    request.setHTTPBody(postData);
-
-    RefPtr<ResourceHandle> loader = ResourceHandle::create(request, this, m_frame, false);
-    loader.get()->ref();
-}
-
-void FrameQtClient::didReceiveResponse(ResourceHandle* job, const ResourceResponse& response)
-{
-    // set mimetype and encoding
-
-    // Assign correct mimetype _before_ calling begin()!
-    m_frame->loader()->setResponseMIMEType(response.mimeType());
-
-    // TODO: Allow user overrides of the encoding...
-    // This calls begin() for us, despite the misleading name
-    if (response.textEncodingName().length()) {
-        // qDebug() << "FrameQtClient: setting encoding to" << response.textEncodingName();
-        m_frame->loader()->setEncoding(response.textEncodingName(), false);
-    } else {
-        m_frame->loader()->begin(job->url());
-    }
-}
-
-void FrameQtClient::didFinishLoading(ResourceHandle* handle)
-{
-    m_frame->loader()->end();
-    m_frame->loader()->checkCompleted();
-    handle->deref();
-}
-
-void FrameQtClient::didFail(ResourceHandle* handle, const ResourceError&)
-{
-    m_frame->loader()->end();
-    m_frame->loader()->checkCompleted();
-    handle->deref();
-}
-
-void FrameQtClient::didReceiveData(ResourceHandle* job, const char* data, int length, int)
-{
-    // Feed with new data
-    m_frame->loader()->write(data, length);
-}
-
-FrameQt* FrameQtClient::traverseNextFrameStayWithin(FrameQt* frame) const
-{
-    return QtFrame(m_frame->tree()->traverseNext(frame));
-}
-
-static int numRequests(Document* document)
-{
-    if (document)
-        return cache()->loader()->numRequests(document->docLoader());
-
-    return 0;
-}
-
-int FrameQtClient::numPendingOrLoadingRequests(bool recurse) const
-{
-    if (!recurse)
-        return numRequests(m_frame->document());
-
-    int num = 0;
-    for (FrameQt* frame = m_frame; frame != 0; frame = traverseNextFrameStayWithin(frame))
-        num += numRequests(frame->document());
-
-    return num;
-}
-
-}
diff --git a/WebCore/page/qt/FrameQtClient.h b/WebCore/page/qt/FrameQtClient.h
deleted file mode 100644 (file)
index c44358c..0000000
+++ /dev/null
@@ -1,71 +0,0 @@
-/*
- * Copyright (C) 2006 Zack Rusin <zack@kde.org>
- * 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.
- */
-
-#ifndef FrameQtClient_h
-#define FrameQtClient_h
-
-#include <wtf/PassRefPtr.h>
-#include "ResourceHandleClient.h"
-
-namespace WebCore {
-
-class String;
-class FrameQt;
-class FormData;
-
-class FrameQtClient : public ResourceHandleClient
-{
-public:
-    FrameQtClient();
-    virtual ~FrameQtClient();
-
-    //local
-    void openURL(const KURL&);
-    void submitForm(const String& method, const KURL&, PassRefPtr<FormData>);
-
-    // FrameQtClient
-    virtual void setFrame(const FrameQt*);
-
-    // ResourceHandleClient
-    virtual void didReceiveResponse(ResourceHandle*, const ResourceResponse&);
-    virtual void didReceiveData(ResourceHandle*, const char*, int, int);
-    virtual void didFinishLoading(ResourceHandle*);
-    virtual void didFail(ResourceHandle*, const ResourceError&);
-private:
-    // Internal helpers
-    FrameQt* traverseNextFrameStayWithin(FrameQt*) const;
-    int numPendingOrLoadingRequests(bool recurse) const;
-
-    FrameQt* m_frame;
-};
-
-}
-
-#endif
-
-// vim: ts=4 sw=4 et
index c234ab2..e279e89 100644 (file)
@@ -61,6 +61,7 @@ class NSURLConnection;
 // WebCoreResourceLoaderImp which avoids doing work in dealloc).
 
 namespace WebCore {
+    class ResourceHandleClient;
 
     class ResourceHandleInternal : Noncopyable {
     public:
index 60c181a..bbd8b04 100644 (file)
@@ -31,6 +31,7 @@
 #include "CString.h"
 #include "FrameQt.h"
 #include "ResourceHandle.h"
+#include "ResourceHandleClient.h"
 #include "ResourceResponse.h"
 #include "ResourceHandleManagerQt.h"
 #include "ResourceHandleInternal.h"
@@ -82,8 +83,8 @@ ResourceHandleManager* ResourceHandleManager::self()
     return s_self;
 }
 
-RequestQt::RequestQt(ResourceHandle* res, FrameQtClient *c)
-    : client(c), resource(res), redirected(false), cancelled(false)
+RequestQt::RequestQt(ResourceHandle* res)
+    : resource(res), redirected(false), cancelled(false)
 {
     setURL(res->url());
     request = QHttpRequestHeader(resource->method(), url.path() + url.query());
@@ -134,7 +135,7 @@ void RequestQt::setURL(const KURL &u)
     qurl = url.url();
 }
 
-void ResourceHandleManager::add(ResourceHandle* resource, FrameQtClient* client)
+void ResourceHandleManager::add(ResourceHandle* resource)
 {
     ASSERT(resource);
 
@@ -144,7 +145,7 @@ void ResourceHandleManager::add(ResourceHandle* resource, FrameQtClient* client)
         return;
     }
 
-    RequestQt* request = new RequestQt(resource, client);
+    RequestQt* request = new RequestQt(resource);
     add(request);
 }
 
index 5ef7033..be37c54 100644 (file)
@@ -56,13 +56,12 @@ struct HostInfo {
 class RequestQt
 {
 public:
-    RequestQt(ResourceHandle*, FrameQtClient*);
+    RequestQt(ResourceHandle*);
     void setURL(const KURL &url);
     // not thread safe, don't use in other threads
     KURL url;
 
     QUrl qurl;
-    FrameQtClient* client;
     ResourceHandle* resource;
 
     // to be used by other threads
@@ -83,7 +82,7 @@ class ResourceHandleManager : public QObject {
 public:
     static ResourceHandleManager* self();
 
-    void add(ResourceHandle*, FrameQtClient*);
+    void add(ResourceHandle*);
     void add(RequestQt* request);
     void cancel(ResourceHandle*);
 
index 9738287..f899538 100644 (file)
@@ -58,7 +58,7 @@ bool ResourceHandle::start(Frame* frame)
     ASSERT(frame);
     FrameQt* f = QtFrame(frame);
 
-    ResourceHandleManager::self()->add(this, f ? f->client() : 0);
+    ResourceHandleManager::self()->add(this);
     return true;
 }
 
index 69520fa..8e8a5a2 100644 (file)
@@ -27,7 +27,6 @@
 #include <qdebug.h>
 
 #include "FrameLoaderClientQt.h"
-#include "FrameQtClient.h"
 #include "FrameQt.h"
 #include "FrameView.h"
 #include "ResourceRequest.h"
@@ -54,7 +53,7 @@ QWebFrame::QWebFrame(QWebPage *parent, QWebFrameData *frameData)
     d->page = parent;
 
     d->frameLoaderClient = new FrameLoaderClientQt();
-    d->frame = new FrameQt(parent->d->page, frameData->ownerElement, new FrameQtClient(), d->frameLoaderClient);
+    d->frame = new FrameQt(parent->d->page, frameData->ownerElement, d->frameLoaderClient);
     d->frameLoaderClient->setFrame(this, d->frame.get());
 
     d->frameView = new FrameView(d->frame.get());
@@ -75,7 +74,7 @@ QWebFrame::QWebFrame(QWebFrame *parent, QWebFrameData *frameData)
     d->page = parent->d->page;
 
     d->frameLoaderClient = new FrameLoaderClientQt();
-    d->frame = new FrameQt(parent->d->page->d->page, frameData->ownerElement, new FrameQtClient(), d->frameLoaderClient);
+    d->frame = new FrameQt(parent->d->page->d->page, frameData->ownerElement, d->frameLoaderClient);
     d->frameLoaderClient->setFrame(this, d->frame.get());
 
     d->frameView = new FrameView(d->frame.get());
index 8223153..f8289ee 100644 (file)
@@ -1,3 +1,11 @@
+2007-01-26  Lars Knoll <lars@trolltech.com>
+
+        Get rid of FrameQtClient. It's unused since we moved over
+        to the loader.
+
+        * Api/qwebframe.cpp:
+        (QWebFrame::QWebFrame):
+
 2007-01-26  Zack Rusin  <zack@kde.org>
 
         Reviewed by Lars