Merge remote-tracking branch 'origin/5.9' into 5.10

 Conflicts:
	src/3rdparty
	src/core/web_contents_adapter.cpp
	src/webengine/api/qquickwebengineprofile.cpp
	src/webenginewidgets/api/qwebengineprofile.cpp
	tests/auto/widgets/qwebenginedownloads/tst_qwebenginedownloads.cpp
	tests/auto/widgets/qwebenginepage/tst_qwebenginepage.cpp

Change-Id: I56c093ebab5ee8b577783ce71761719159cd3ddd

1 files changed, 17 insertions, 4 deletions

diff --git a/tests/auto/widgets/qwebenginedownloads/tst_qwebenginedownloads.cpp b/tests/auto/widgets/qwebenginedownloads/tst_qwebenginedownloads.cpp
index 7094c8e4b..83e8d1101 100644
--- a/tests/auto/widgets/qwebenginedownloads/tst_qwebenginedownloads.cpp
+++ b/tests/auto/widgets/qwebenginedownloads/tst_qwebenginedownloads.cpp
@@ -431,6 +431,13 @@ void tst_QWebEngineDownloads::downloadLink()
 void tst_QWebEngineDownloads::downloadTwoLinks()
 {
     HttpServer server;
+    QSignalSpy requestSpy(&server, &HttpServer::newRequest);
+    QList<HttpReqRep*> results;
+    connect(&server, &HttpServer::newRequest, [&](HttpReqRep *rr) {
+        rr->setParent(nullptr);
+        results.append(rr);
+    });
+
     QWebEngineProfile profile;
     QWebEnginePage page(&profile);
     QWebEngineView view;
@@ -438,7 +445,8 @@ void tst_QWebEngineDownloads::downloadTwoLinks()
 
     view.load(server.url());
     view.show();
-    auto indexRR = waitForRequest(&server);
+    QTRY_COMPARE(requestSpy.count(), 1);
+    std::unique_ptr<HttpReqRep> indexRR(results.takeFirst());
     QVERIFY(indexRR);
     QCOMPARE(indexRR->requestMethod(), QByteArrayLiteral("GET"));
     QCOMPARE(indexRR->requestPath(), QByteArrayLiteral("/"));
@@ -449,19 +457,24 @@ void tst_QWebEngineDownloads::downloadTwoLinks()
     QVERIFY(waitForSignal(&page, &QWebEnginePage::loadFinished, [&](bool ok){ loadOk = ok; }));
     QVERIFY(loadOk);
 
-    auto favIconRR = waitForFaviconRequest(&server);
+    QTRY_COMPARE(requestSpy.count(), 2);
+    std::unique_ptr<HttpReqRep> favIconRR(results.takeFirst());
     QVERIFY(favIconRR);
+    favIconRR->setResponseStatus(404);
+    favIconRR->sendResponse();
 
     QTRY_VERIFY(view.focusWidget());
     QWidget *renderWidget = view.focusWidget();
     QTest::mouseClick(renderWidget, Qt::LeftButton, {}, QPoint(10, 10));
     QTest::mouseClick(renderWidget, Qt::LeftButton, {}, QPoint(10, 30));
 
-    auto file1RR = waitForRequest(&server);
+    QTRY_VERIFY(requestSpy.count() >= 3);
+    std::unique_ptr<HttpReqRep> file1RR(results.takeFirst());
     QVERIFY(file1RR);
     QCOMPARE(file1RR->requestMethod(), QByteArrayLiteral("GET"));
     QCOMPARE(file1RR->requestPath(), QByteArrayLiteral("/file1"));
-    auto file2RR = waitForRequest(&server);
+    QTRY_COMPARE(requestSpy.count(), 4);
+    std::unique_ptr<HttpReqRep> file2RR(results.takeFirst());
     QVERIFY(file2RR);
     QCOMPARE(file2RR->requestMethod(), QByteArrayLiteral("GET"));
     QCOMPARE(file2RR->requestPath(), QByteArrayLiteral("/file2"));