Merge remote-tracking branch 'origin/5.13' into dev

 Conflicts:
	src/webchannel/qmetaobjectpublisher.cpp

Change-Id: I853b8fc7270847c1a9cdd862dff278ddb2533e92

2 files changed, 11 insertions, 1 deletions

diff --git a/src/webchannel/qmetaobjectpublisher.cpp b/src/webchannel/qmetaobjectpublisher.cpp
index 9f5e9cd..102cbda 100644
--- a/src/webchannel/qmetaobjectpublisher.cpp
+++ b/src/webchannel/qmetaobjectpublisher.cpp
@@ -781,7 +781,10 @@ QJsonValue QMetaObjectPublisher::wrapResult(const QVariant &result, QWebChannelA
 #endif
     } else if (result.canConvert<QVariantList>()) {
         // recurse and potentially wrap contents of the array
-        return wrapList(result.toList(), transport);
+        // *don't* use result.toList() as that *only* works for QVariantList and QStringList!
+        // Also, don't use QSequentialIterable (yet), since that seems to trigger QTBUG-42016
+        // in certain cases.
+        return wrapList(result.value<QVariantList>(), transport);
     } else if (result.canConvert<QVariantMap>()) {
         // recurse and potentially wrap contents of the map
         return wrapMap(result.toMap(), transport);
diff --git a/tests/auto/webchannel/tst_webchannel.cpp b/tests/auto/webchannel/tst_webchannel.cpp
index 4e622c7..ffaebba 100644
--- a/tests/auto/webchannel/tst_webchannel.cpp
+++ b/tests/auto/webchannel/tst_webchannel.cpp
@@ -827,6 +827,13 @@ void TestWebChannel::testWrapValues()
         QVERIFY(value.isDouble());
         QCOMPARE(value.toInt(), (int) flags);
     }
+    {
+        QVector<int> vec{1, 2, 3};
+        QVariant variant = QVariant::fromValue(vec);
+        QJsonValue value = channel.d_func()->publisher->wrapResult(variant, m_dummyTransport);
+        QVERIFY(value.isArray());
+        QCOMPARE(value.toArray(), QJsonArray({1, 2, 3}));
+    }
 }
 
 void TestWebChannel::testWrapObjectWithMultipleTransports()