Use range-based for instead of Q_FOREACH

Change-Id: Ibf016b795ff98fddfa29fb5dc63924a2d2159d71
Reviewed-by: Michal Klocek <michal.klocek@qt.io>

2 files changed, 4 insertions, 3 deletions

diff --git a/src/webengine/api/qquickwebengineprofile.cpp b/src/webengine/api/qquickwebengineprofile.cpp
index 3c22dba98..6c7985e3f 100644
--- a/src/webengine/api/qquickwebengineprofile.cpp
+++ b/src/webengine/api/qquickwebengineprofile.cpp
@@ -167,7 +167,7 @@ QQuickWebEngineProfilePrivate::~QQuickWebEngineProfilePrivate()
         m_browserContextAdapter->removeClient(this);
     }
 
-    Q_FOREACH (QQuickWebEngineDownloadItem *download, m_ongoingDownloads) {
+    for (QQuickWebEngineDownloadItem *download : qAsConst(m_ongoingDownloads)) {
         if (download)
             download->cancel();
     }
diff --git a/src/webengine/api/qquickwebengineview.cpp b/src/webengine/api/qquickwebengineview.cpp
index 32df81538..ba7377c2c 100644
--- a/src/webengine/api/qquickwebengineview.cpp
+++ b/src/webengine/api/qquickwebengineview.cpp
@@ -768,7 +768,7 @@ void QQuickWebEngineViewPrivate::initializationFinished()
     if (devToolsView && devToolsView->d_ptr->adapter)
         adapter->openDevToolsFrontend(devToolsView->d_ptr->adapter);
 
-    Q_FOREACH (QQuickWebEngineScript *script, m_userScripts)
+    for (QQuickWebEngineScript *script : qAsConst(m_userScripts))
         script->d_func()->bind(browserContextAdapter()->userResourceController(), adapter.data());
 
     if (!m_isBeingAdopted)
@@ -1371,7 +1371,8 @@ void QQuickWebEngineView::fullScreenCancelled()
 void QQuickWebEngineView::geometryChanged(const QRectF &newGeometry, const QRectF &oldGeometry)
 {
     QQuickItem::geometryChanged(newGeometry, oldGeometry);
-    Q_FOREACH(QQuickItem *child, childItems()) {
+    const QList<QQuickItem *> children = childItems();
+    for (QQuickItem *child : children) {
         if (qobject_cast<RenderWidgetHostViewQtDelegateQuick *>(child))
             child->setSize(newGeometry.size());
     }