[WPE][Qt] loadingChanged signal parameter is unusable
authorphiln@webkit.org <philn@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Aug 2019 08:29:41 +0000 (08:29 +0000)
committerphiln@webkit.org <philn@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 30 Aug 2019 08:29:41 +0000 (08:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=201301

Reviewed by Carlos Garcia Campos.

Source/WebKit:

* UIProcess/API/wpe/qt/WPEQtView.h: Explicitely name signal
parameter. It is a runtime requirement for QML.

Tools:

* MiniBrowser/wpe/qt/main.qml: Add a basic loadingChange signal handler.
* Scripts/webkitpy/style/checker.py: White-list WPEQtView.h for
readablity/parameter_name code styl checker rule.

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/API/wpe/qt/WPEQtView.h
Tools/ChangeLog
Tools/MiniBrowser/wpe/qt/main.qml
Tools/Scripts/webkitpy/style/checker.py

index 1b6fdf4..928a0b3 100644 (file)
@@ -1,3 +1,13 @@
+2019-08-30  Philippe Normand  <pnormand@igalia.com>
+
+        [WPE][Qt] loadingChanged signal parameter is unusable
+        https://bugs.webkit.org/show_bug.cgi?id=201301
+
+        Reviewed by Carlos Garcia Campos.
+
+        * UIProcess/API/wpe/qt/WPEQtView.h: Explicitely name signal
+        parameter. It is a runtime requirement for QML.
+
 2019-08-29  Keith Rollin  <krollin@apple.com>
 
         Remove HAVE_PASSKIT_GRANULAR_ERRORS conditional
index 1af8e1c..0f282bd 100644 (file)
@@ -77,7 +77,7 @@ Q_SIGNALS:
     void webViewCreated();
     void urlChanged();
     void titleChanged();
-    void loadingChanged(WPEQtViewLoadRequest*);
+    void loadingChanged(WPEQtViewLoadRequest* loadRequest);
     void loadProgressChanged();
 
 protected:
index 09bd657..449766e 100644 (file)
@@ -1,3 +1,14 @@
+2019-08-30  Philippe Normand  <pnormand@igalia.com>
+
+        [WPE][Qt] loadingChanged signal parameter is unusable
+        https://bugs.webkit.org/show_bug.cgi?id=201301
+
+        Reviewed by Carlos Garcia Campos.
+
+        * MiniBrowser/wpe/qt/main.qml: Add a basic loadingChange signal handler.
+        * Scripts/webkitpy/style/checker.py: White-list WPEQtView.h for
+        readablity/parameter_name code styl checker rule.
+
 2019-08-29  Keith Rollin  <krollin@apple.com>
 
         Update .xcconfig symbols to reflect the current set of past and future product versions.
index a405d76..67420a4 100644 (file)
@@ -16,5 +16,9 @@ Window {
         onTitleChanged: {
             main_window.title = title;
         }
+        onLoadingChanged: {
+            if (loadRequest.errorString)
+                console.log('WPEView error: ' + loadRequest.errorString)
+        }
     }
 }
index dc088e9..946ca2f 100644 (file)
@@ -154,6 +154,14 @@ _PATH_RULES_SPECIFIER = [
      ["-readability/naming"]),
 
     ([
+      # The WPEQtView class can't rely on the readability/parameter_name rule,
+      # because omitting parameter names for QML signals leads to runtime
+      # errors.
+      os.path.join('Source', 'WebKit', 'UIProcess', 'API', 'wpe', 'qt', 'WPEQtView.h'),
+    ],
+    ["-readability/parameter_name"]),
+
+    ([
       # The GTK+ and WPE APIs use upper case, underscore separated, words in
       # certain types of enums (e.g. signals, properties).
       os.path.join('Source', 'JavaScriptCore', 'API', 'glib'),