Changeset 96101 in webkit

Timestamp:

Sep 27, 2011 5:58:02 AM (13 years ago)

Author:

alexis.menard@openbossa.org

Message:

[Qt][WK2] API fixes for QML, the signal parameters needs to be named.
​https://bugs.webkit.org/show_bug.cgi?id=68889

Reviewed by Andreas Kling.

Signal parameters needs to be explicitly named in QML to be accessible.
This patch fix this problem.

• UIProcess/API/qt/qdesktopwebview.h:
• UIProcess/API/qt/qtouchwebpage.h:

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/API/qt/qdesktopwebview.h (modified) (1 diff)
• UIProcess/API/qt/qtouchwebpage.h (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2011-09-27  Alexis Menard  <alexis.menard@openbossa.org>
+
+        [Qt][WK2] API fixes for QML, the signal parameters needs to be named.
+        https://bugs.webkit.org/show_bug.cgi?id=68889
+
+        Reviewed by Andreas Kling.
+
+        Signal parameters needs to be explicitly named in QML to be accessible.
+        This patch fix this problem.
+
+        * UIProcess/API/qt/qdesktopwebview.h:
+        * UIProcess/API/qt/qtouchwebpage.h:
+
 2011-09-27  Alexis Menard  <alexis.menard@openbossa.org>
 

trunk/Source/WebKit2/UIProcess/API/qt/qdesktopwebview.h

@@ -82 +82 @@
 
 Q_SIGNALS:
-    void titleChanged(const QString&);
-    void statusBarMessageChanged(const QString&);
+    void titleChanged(const QString& title);
+    void statusBarMessageChanged(const QString& message);
     void loadStarted();
     void loadSucceeded();
     void loadFailed(QDesktopWebView::ErrorType errorType, int errorCode, const QUrl& url);
     void loadProgressChanged(int progress);
-    void urlChanged(const QUrl&);
+    void urlChanged(const QUrl& url);
 
 protected:

trunk/Source/WebKit2/UIProcess/API/qt/qtouchwebpage.h

@@ -66 +66 @@
 
 Q_SIGNALS:
-    void urlChanged(const QUrl&);
-    void titleChanged(const QString&);
+    void urlChanged(const QUrl& url);
+    void titleChanged(const QString& title);
     void loadStarted();
     void loadSucceeded();