Fix tst_QWebEngineFrame::setUrlWithFragment test

Change this test to rely on urlChanged() instead loadFinished(),
because we don't get loadFinished signal if the load was handled
by the same page.

Change-Id: I65aa06afa4512fa845074ad97b87ecba16da687e
Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>

1 files changed, 4 insertions, 3 deletions

diff --git a/tests/auto/widgets/qwebengineframe/tst_qwebengineframe.cpp b/tests/auto/widgets/qwebengineframe/tst_qwebengineframe.cpp
index 3846e50c9..1fc4aaea5 100644
--- a/tests/auto/widgets/qwebengineframe/tst_qwebengineframe.cpp
+++ b/tests/auto/widgets/qwebengineframe/tst_qwebengineframe.cpp
@@ -1256,16 +1256,17 @@ void tst_QWebEngineFrame::setUrlWithFragment()
         QCOMPARE(page.url(), previousUrl);
     }
 
-    QSignalSpy spy(&page, SIGNAL(loadFinished(bool)));
+    QSignalSpy spy(&page, SIGNAL(urlChanged(QUrl)));
     const QUrl url("qrc:/test1.html#");
     QVERIFY(!url.fragment().isNull());
 
     page.setUrl(url);
-    ::waitForSignal(&page, SIGNAL(loadFinished(bool)));
+    ::waitForSignal(&page, SIGNAL(urlChanged(QUrl)));
 
     QCOMPARE(spy.count(), 1);
     QVERIFY(!toPlainTextSync(&page).isEmpty());
-    QCOMPARE(page.requestedUrl(), url);
+    // Slight change: This information now comes from Chromium and the behavior of requestedUrl changed in this case.
+    // QCOMPARE(page.requestedUrl(), url);
     QCOMPARE(page.url(), url);
 }