Reviewed by Zack.
authorstaikos <staikos@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 10 Dec 2006 06:54:03 +0000 (06:54 +0000)
committerstaikos <staikos@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 10 Dec 2006 06:54:03 +0000 (06:54 +0000)
        Make it not crash, and make it set the title on Qt platform.

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

WebCore/ChangeLog
WebCore/loader/qt/FrameLoaderClientQt.cpp
WebCore/loader/qt/FrameLoaderQt.cpp

index 02725990ac02ff5e74335dc6acaa91023a668c13..1c5b6df2439b981908e05ad5b7837b8f8b8de25b 100644 (file)
@@ -1,3 +1,17 @@
+2006-12-09  George Staikos  <staikos@kde.org>
+
+        Reviewed by Zack.
+
+        Make it not crash, and make it set the title on Qt platform.
+
+        * loader/qt/FrameLoaderClientQt.cpp:
+        (WebCore::FrameLoaderClientQt::hasWebView):
+        (WebCore::FrameLoaderClientQt::hasFrameView):
+        (WebCore::FrameLoaderClientQt::didChangeTitle):
+        * loader/qt/FrameLoaderQt.cpp:
+        (WebCore::FrameLoader::setTitle):
+        (WebCore::FrameLoader::checkLoadCompleteForThisFrame):
+
 2006-12-10  Zack Rusin  <zack@kde.org>
 
         Renaming FrameQtClientDefault to FrameQtClient.
index e157e25a8efd014a86dd86bbb759ea0d3a30ddfc..95c759f24f6b1727a13c3feee0968af6577b59d9 100644 (file)
@@ -28,6 +28,7 @@
  
 #include "config.h"
 #include "FrameLoaderClientQt.h"
+#include "DocumentLoader.h"
 
 #define notImplemented() do { fprintf(stderr, "FIXME: UNIMPLEMENTED: %s:%d\n", __FILE__, __LINE__); } while(0)
 
@@ -64,14 +65,14 @@ void FrameLoaderClientQt::deref()
 bool FrameLoaderClientQt::hasWebView() const
 {
     notImplemented();
-    return false;
+    return true;
 }
 
 
 bool FrameLoaderClientQt::hasFrameView() const
 {
     notImplemented();
-    return false;
+    return true;
 }
 
 
@@ -378,9 +379,9 @@ void FrameLoaderClientQt::willChangeTitle(DocumentLoader*)
 }
 
 
-void FrameLoaderClientQt::didChangeTitle(DocumentLoader*)
+void FrameLoaderClientQt::didChangeTitle(DocumentLoader *l)
 {
-    notImplemented();
+    setTitle(l->title(), l->URL());
 }
 
 
index 3081b795610dae9db5b4ada6217204822c237552..53907b2df7b8c3fd3a448011c30503c6155a1d2f 100644 (file)
@@ -117,8 +117,7 @@ void FrameLoader::urlSelected(const FrameLoadRequest& frameLoadRequest, Event*)
 
 void FrameLoader::setTitle(const String& title)
 {
-    if (m_frame->view() && m_frame->view()->parentWidget())
-        m_frame->view()->parentWidget()->setWindowTitle(title);
+    client()->setTitle(title, URL());
 }
 
 Frame* FrameLoader::createFrame(const KURL& url, const String& name, Element* ownerElement, const String& referrer)
@@ -182,7 +181,7 @@ void FrameLoader::checkLoadCompleteForThisFrame()
 
     case FrameStateCommittedPage: {
         DocumentLoader* dl = m_documentLoader.get();
-        if (dl->isLoadingInAPISense())
+        if (!dl || dl->isLoadingInAPISense())
             return;
 
         markLoadComplete();