Fix the Qt build.
authorhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 18 Mar 2008 15:20:49 +0000 (15:20 +0000)
committerhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 18 Mar 2008 15:20:49 +0000 (15:20 +0000)
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@31126 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/platform/qt/MainThreadQt.cpp
WebKit/qt/Api/qwebpluginfactory.cpp
WebKit/qt/ChangeLog

index d5f0dc7..05a2714 100644 (file)
@@ -1,3 +1,13 @@
+2008-03-18  Simon Hausmann  <hausmann@webkit.org>
+
+        Reviewed by Holger.
+
+        Fix the Qt build and clean up the invoke mechanism for dispatching
+        functions in the main thread by using QMetaObject::invokeMethod
+        instead of posting a null event.
+
+        * platform/qt/MainThreadQt.cpp:
+
 2008-03-18  Rodney Dawes  <dobey@wayofthemonkey.com>
 
         Fix Mac build from commit of r31123.
index 35f08ae..51ccdbe 100644 (file)
@@ -42,8 +42,8 @@ class MainThreadInvoker : public QObject {
 public:
     MainThreadInvoker();
 
-protected:
-    bool event(QEvent*);
+private Q_SLOTS:
+    void dispatch();
 };
 
 MainThreadInvoker::MainThreadInvoker()
@@ -51,7 +51,7 @@ MainThreadInvoker::MainThreadInvoker()
     moveToThread(QCoreApplication::instance()->thread());
 }
 
-bool MainThreadInvoker::event(QEvent*)
+void MainThreadInvoker::dispatch()
 {
     dispatchFunctionsFromMainThread();
 }
@@ -61,7 +61,7 @@ Q_GLOBAL_STATIC(MainThreadInvoker, webkit_main_thread_invoker)
 
 void scheduleDispatchFunctionsOnMainThread()
 {
-    QCoreApplication::postEvent(webkit_main_thread_invoker(), 0);
+    QMetaObject::invokeMethod(webkit_main_thread_invoker(), "dispatch", Qt::QueuedConnection);
 }
 
 }
index cb5d455..6f2a6d5 100644 (file)
@@ -16,8 +16,10 @@ void QWebPluginFactory::refreshPlugins()
 
 bool QWebPluginFactory::extension(Extension extension, const ExtensionOption *option, ExtensionReturn *output)
 {
+    return false;
 }
 
 bool QWebPluginFactory::supportsExtension(Extension extension) const
 {
+    return false;
 }
index ab2007c..b0220de 100644 (file)
@@ -1,3 +1,12 @@
+2008-03-18  Simon Hausmann  <hausmann@webkit.org>
+
+        Reviewed by Holger.
+
+        Fix the Qt build. Don't return void in non-void functions.
+
+        * Api/qwebpluginfactory.cpp:
+        (QWebPluginFactory::extension):
+
 2008-03-16  Thiago Macieira  <thiago.macieira@trolltech.com>
 
         Reviewed by Simon.