2011-02-06 Andreas Kling <kling@webkit.org>
authorandreas.kling@nokia.com <andreas.kling@nokia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 6 Feb 2011 15:19:26 +0000 (15:19 +0000)
committerandreas.kling@nokia.com <andreas.kling@nokia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 6 Feb 2011 15:19:26 +0000 (15:19 +0000)
        Reviewed by Antonio Gomes.

        [Qt] MiniBrowser: Clean up handling of titleChanged() signal
        https://bugs.webkit.org/show_bug.cgi?id=53869

        * MiniBrowser/qt/BrowserView.cpp:
        (BrowserView::BrowserView): Remove unnecessary titleChanged() handling.
        * MiniBrowser/qt/BrowserWindow.cpp:
        (BrowserWindow::BrowserWindow): Connect titleChanged() directly to setWindowTitle().
        * MiniBrowser/qt/BrowserWindow.h:

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

Tools/ChangeLog
Tools/MiniBrowser/qt/BrowserView.cpp
Tools/MiniBrowser/qt/BrowserWindow.cpp
Tools/MiniBrowser/qt/BrowserWindow.h

index beccb512bc66ee6e7fc32d5776de9f04bf130da5..6ae58a52828588ef4fdb56c42558aa7d9360917c 100644 (file)
@@ -1,3 +1,16 @@
+2011-02-06  Andreas Kling  <kling@webkit.org>
+
+        Reviewed by Antonio Gomes.
+
+        [Qt] MiniBrowser: Clean up handling of titleChanged() signal
+        https://bugs.webkit.org/show_bug.cgi?id=53869
+
+        * MiniBrowser/qt/BrowserView.cpp:
+        (BrowserView::BrowserView): Remove unnecessary titleChanged() handling.
+        * MiniBrowser/qt/BrowserWindow.cpp:
+        (BrowserWindow::BrowserWindow): Connect titleChanged() directly to setWindowTitle().
+        * MiniBrowser/qt/BrowserWindow.h:
+
 2011-02-06  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Dan Bernstein.
index 18d262bd08e29bd1f732b73d13ce12d244b32f6c..7d6426eb8621e63613aba2131e1cd78708f25232 100644 (file)
@@ -41,8 +41,6 @@ BrowserView::BrowserView(QGraphicsWKView::BackingStoreType backingStoreType, QWK
     setFrameShape(QFrame::NoFrame);
     setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff);
     setVerticalScrollBarPolicy(Qt::ScrollBarAlwaysOff);
-
-    connect(m_item, SIGNAL(titleChanged(QString)), this, SLOT(setWindowTitle(QString)));
 }
 
 void BrowserView::resizeEvent(QResizeEvent* event)
index ab4ec3666899517a0e7676f5b070d183c4645945..bfd7f957c88d8597ff4fdaf1d0382f8de1274948 100644 (file)
@@ -50,7 +50,7 @@ BrowserWindow::BrowserWindow(QWKContext* context)
     setAttribute(Qt::WA_DeleteOnClose);
 
     connect(m_browser->view(), SIGNAL(loadProgress(int)), SLOT(loadProgress(int)));
-    connect(m_browser->view(), SIGNAL(titleChanged(const QString&)), SLOT(titleChanged(const QString&)));
+    connect(m_browser->view(), SIGNAL(titleChanged(const QString&)), SLOT(setWindowTitle(const QString&)));
     connect(m_browser->view(), SIGNAL(urlChanged(const QUrl&)), SLOT(urlChanged(const QUrl&)));
 
     this->setCentralWidget(m_browser);
@@ -186,11 +186,6 @@ void BrowserWindow::loadProgress(int progress)
     m_addressBar->setPalette(pallete);
 }
 
-void BrowserWindow::titleChanged(const QString& title)
-{
-    setWindowTitle(title);
-}
-
 void BrowserWindow::urlChanged(const QUrl& url)
 {
     m_addressBar->setText(url.toString());
index b43e1a4abc9f02094fd0030d0153081833f78cde..f0a642ea6a667c0a665b320e2161ee458a695b8d 100644 (file)
@@ -57,7 +57,6 @@ signals:
 protected slots:
     void changeLocation();
     void loadProgress(int progress);
-    void titleChanged(const QString&);
     void urlChanged(const QUrl&);
     void openFile();