Add zoomFactorChanged signal to QWebEnginePage and QQuickWebEngineView

Added a signal that fires whenever the zoom factor for a given page
changes, including both via key combinations, and when using the
existing zoom API.

This already existed in Quick, but was both undocumented and only
half-implemented (zooming via keyboard/mouse wouldn't trigger it).
Thus, the Quick documentation was changed to list it as a 6.8 addition.

Fixes: QTBUG-111041
Change-Id: I91eb7c9bf0d17d99956784841c5c86640554ad1c
Reviewed-by: Allan Sandfeld Jensen <[email protected]>

1 files changed, 6 insertions, 1 deletions

diff --git a/src/webenginequick/api/qquickwebengineview.cpp b/src/webenginequick/api/qquickwebengineview.cpp
index 85c5e1a73..5a68cf687 100644
--- a/src/webenginequick/api/qquickwebengineview.cpp
+++ b/src/webenginequick/api/qquickwebengineview.cpp
@@ -572,6 +572,12 @@ void QQuickWebEngineViewPrivate::iconChanged(const QUrl &url)
     QTimer::singleShot(0, q, &QQuickWebEngineView::iconChanged);
 }
 
+void QQuickWebEngineViewPrivate::zoomFactorChanged(qreal factor)
+{
+    Q_Q(QQuickWebEngineView);
+    Q_EMIT q->zoomFactorChanged(factor);
+}
+
 void QQuickWebEngineViewPrivate::loadProgressChanged(int progress)
 {
     Q_Q(QQuickWebEngineView);
@@ -1225,7 +1231,6 @@ void QQuickWebEngineView::setZoomFactor(qreal arg)
         d->adapter->setZoomFactor(arg);
         // MEMO: should reset if factor was not applied due to being invalid
         d->m_zoomFactor = zoomFactor();
-        emit zoomFactorChanged(d->m_zoomFactor);
     } else {
         d->m_zoomFactor = arg;
     }